Claude/subagents deploy 142 148 dhe3mq#125
Merged
Merged
Conversation
… PLAN.md long-term sections to feature-ideas.md - Delete the fully-shipped "index doctor --fix Generalization & Multi-Profile Ephemeral-Job Parity" idea (Phases 134/135); trim "Public Repo Sharing: Throttle/Rate-Limit Unification" down to its still-open half (Phase 133 already shipped the policy-extraction part). Fix features.md's stale "1380 tests" banner to the current count. - Remove the tools lsp --tcp transport entirely (Phase 149): it had no auth mechanism at all and nothing exercised it in the test suite. Updates deprecations.md, README, features.md, parity.md, and CLAUDE.md accordingly, plus a changeset. - Move docs/PLAN.md's "Long-Term Investments" and "Non-goals for now" sections into docs/feature-ideas.md (Plugin API entry already existed there; added Scale & Performance Notes and a Non-goals section for the rest). pnpm build && pnpm test green (1572 passed, 22 skipped).
…-deploy-142-148-dhe3mq Doc pruning, --tcp removal, and PLAN.md long-term-section move
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Task A (prune feature-ideas.md): ✅ fully done by the subagent — deleted the fully-shipped index doctor --fix/ephemeral-job entry, trimmed the throttle-unification entry to its remaining open half, fixed the stale "1380 tests" banner to the real count (1594).
Task B (--tcp removal): subagent finished the code removal, deprecations.md, README/features.md/parity.md updates, and the new Phase 149 PLAN.md entry — but it never actually ran the missing pieces it described: I added the missing .changeset/remove-lsp-tcp-transport.md and updated CLAUDE.md's two remaining --tcp mentions (a stale "Known gaps" row and two architecture-diagram lines) that it wrote into PLAN.md's "Files touched" list but hadn't touched.
Task C (move PLAN.md's Long-Term Investments/Non-goals into feature-ideas.md): not started by the subagent at all — I did this myself: removed both sections + their TOC anchors from PLAN.md, folded the Plugin API cross-reference into its existing feature-ideas.md entry, and added new "Scale & Performance Notes" and "Non-goals (Deliberately Out of Scope)" sections there.