Skip to content

docs(roadmap): add #413 — acp json leaks internal tracking numbers as public fields#2957

Open
code-yeongyu wants to merge 1 commit intomainfrom
docs/roadmap-413-acp-leaks-internal-tracking-fields
Open

docs(roadmap): add #413 — acp json leaks internal tracking numbers as public fields#2957
code-yeongyu wants to merge 1 commit intomainfrom
docs/roadmap-413-acp-leaks-internal-tracking-fields

Conversation

@code-yeongyu
Copy link
Copy Markdown
Collaborator

Pinpoint #413

acp --output-format json exposes internal backlog references in the public JSON contract:

{
  "discoverability_tracking": "ROADMAP #64a",
  "tracking": "ROADMAP #76",
  "message": "ACP/Zed editor integration is not implemented...",
  "status": "discoverability_only",
  "supported": false,
  ...
}

Two fields (discoverability_tracking, tracking) are verbatim internal backlog ticket numbers that should never appear in a public machine-readable contract. message is implementation-note prose that serves as the sole machine-classifiable signal.

ROADMAP-only PR. No source/test changes.

@Yeachan-Heo
Copy link
Copy Markdown
Contributor

Review verdict: APPROVE

Inspected exact diff against origin/main: docs-only ROADMAP.md change (+2), no source/test/runtime contract files touched. CI is green and merge state is CLEAN.

The #413 pinpoint is concrete and actionable: it names the exact command (acp --output-format json), observed public JSON keys (discoverability_tracking, tracking, message), and the contract problem (internal ROADMAP/backlog references leaking into machine-readable output).

Non-duplicative: nearby ACP/help JSON items cover discoverability/unsupported ACP support or help-format behavior; this specifically tracks public JSON leakage of internal tracking fields. Minor wording nuance: ACP JSON already has status, supported, and serve_alias_only, so message is not literally the only classifier, but it is the only detailed reason/status explanation. Not a blocker for a roadmap pinpoint.

Merge risk: low. Blockers: none.


[repo owner's gaebal-gajae (clawdbot) 🦞]

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.

2 participants