Skip to content

Remove legacy compatibility code; rewrite README - #6

Merged
kchandan merged 2 commits into
mainfrom
cleanup-legacy
Oct 23, 2026
Merged

kchandan merged 2 commits into
mainfrom
cleanup-legacy

Conversation

@kchandan

Copy link
Copy Markdown
Member

Removed (legacy left from the rename and the earlier boto3-only design)

  • labctl console script. Breaking for anyone who used it (shipped in 0.1.0 and 0.2.1). labctl is another project's name on PyPI, and installing labagent put a second command with that name on users' PATH. Use labagent.
  • LABCTL_* environment fallback: only LABAGENT_* is read (never shipped in a release).
  • The four silently ignored legacy config keys (group_name, policy_name, lambda_role_name, policy_file): they are now rejected as unknown keys (never shipped in a release).
  • Unused deleters.order_of helper and an unused import in run.py.
  • Tests that protected the shims now pin the new behavior; a new test asserts labagent is the only installed command.

README

Rewritten short (188 to 84 lines): features list, architecture diagram, deployment-modes table, FAQ and marketing sections cut; badges unchanged. Removed a claim for something that does not exist (AWS labs running in a private CI repo with GitHub OIDC) and a stale test count. Added the rag-ide pack and the local RAG lab, marked unreleased.

Checked

  • 249 tests pass, PII scanner clean, dead-code scan (vulture, 80%) clean.
  • Nothing in the maintainers' private configs, scripts or run logs uses the labctl command, LABCTL_* variables or the legacy keys.

Left on purpose

student_domain config field: no code reads it, but existing configs set it, so removing it would break them. Candidate for a later, deliberate change.

🤖 Generated with Claude Code

kchandan and others added 2 commits October 23, 2026 15:23
- drop the labctl console script (another project's name on PyPI; it put a
  second command on users' PATH)
- drop the LABCTL_* environment fallback and the silently ignored legacy config
  keys (group_name, policy_name, lambda_role_name, policy_file): unknown config
  keys are now an error instead of being ignored
- remove the unused deleters.order_of helper and an unused import in run.py
- tests that protected the shims now pin the new behaviour

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Cut the feature list, ASCII architecture, deployment-modes table, FAQ and marketing
sections. Remove claims for things that do not exist (AWS labs running in a private
CI repo with OIDC; a stale test count). Add the rag-ide pack, its three platforms
and the local RAG lab, marked unreleased. Badges unchanged.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@kchandan
kchandan merged commit df1dced into main Oct 23, 2026
3 checks passed
@kchandan
kchandan deleted the cleanup-legacy branch October 23, 2026 19:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant