chore: remove local repository map command - #109
Merged
Merged
Conversation
Co-Authored-By: coreplane-switchboard[bot] <318072483+coreplane-switchboard[bot]@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
LGTM: The unsupported map surface and its mapping-only machinery are removed completely without disturbing setup or MCP behavior.
Note
Approved · head ecc024c · no findings
Full review
No findings; the removal is complete and preserves unrelated setup and MCP behavior.
Contributor
Author
|
Rollout hold: do not merge or publish this CLI removal until the coordinated polylanedotcom setup-script change is merged and confirmed live. After the CLI release is published, remove/deploy the nominal MCP/server mapping surface last. |
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
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.
The CLI no longer exposes
polylane mapor carries machinery used only to launch local repository mapping. This keeps setup focused on connections and removes a dead fallback coordinated with the installer and platform cleanup.Why: Local repository mapping backed an installer path that is being removed across Polylane. Leaving the command and headless-agent recipes behind would preserve an unsupported entry point and unnecessary maintenance surface.
Rollout hold: Do not merge or publish this PR until the polylanedotcom installer change is merged and live. The nominal MCP/server removal follows this CLI release.
Where to look
map. ⚠ A stale registration would keep the removed command reachable.mapas removed and excludes it from resource groups.Feedback wanted: Check that no remaining setup behavior depends on the removed headless-run metadata and that the command surface is fully closed.
Risk: A missed reference could expose stale help or break agent setup. Roll back this single commit if coordinated installer/platform removal is delayed.
Verified: 87 focused tests, scoped ESLint, diff check, and removal-residue check passed; full CI is pending.
Decisions (1)
polylane mapwas removed, so retaining them would create dead API surface.Validation (4 criteria)
For agents
This is one part of a coordinated three-repository removal:
polylanedotcomremoves the installer fallback andnominalremoves the platform mapping APIs and storage. Review this PR for CLI-local completeness without expecting those sibling changes in this diff.