Skip to content

docs(roadmap): add #410 — agents/skills/mcp list count-field naming drift#2954

Open
code-yeongyu wants to merge 2 commits intomainfrom
docs/roadmap-410-list-envelope-count-naming-drift
Open

docs(roadmap): add #410 — agents/skills/mcp list count-field naming drift#2954
code-yeongyu wants to merge 2 commits intomainfrom
docs/roadmap-410-list-envelope-count-naming-drift

Conversation

@code-yeongyu
Copy link
Copy Markdown
Collaborator

Pinpoint #410

agents list, skills list, and mcp list use three different count-field names and divergent envelope schemas:

command count field summary obj working_directory status field
agents list count yes yes no
skills list absent yes no no
mcp list configured_servers no yes yes

Concrete naming-drift facet of #403. Orchestration cannot extract cardinality with one pattern.

ROADMAP-only PR. No source/test changes.

@Yeachan-Heo
Copy link
Copy Markdown
Contributor

Review verdict: REQUEST_CHANGES

I checked the exact diff against origin/main and the PR is docs-only (ROADMAP.md | 2 ++). CI is green and merge risk is low. The #410 pinpoint itself is concrete and actionable: it names the affected commands (agents list, skills list, mcp list), identifies the count/envelope drift, and gives a reasonable fix shape.

Blocker: the new text says this is part of a broader inconsistency “noted in #403”, but this ROADMAP.md branch has no item 403 / no #403 reference to resolve. That makes the roadmap cross-reference misleading.

Minimal fix: remove that reference or replace it with non-numbered wording, e.g. “This is a concrete naming-drift facet of the broader list-envelope inconsistency…”

After that narrow docs fix, I would approve.


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

@code-yeongyu
Copy link
Copy Markdown
Collaborator Author

Fixed: removed forward reference to unmerged #403 in the ROADMAP #410 entry text. New wording is self-contained — no cross-reference to in-flight PRs.

@Yeachan-Heo
Copy link
Copy Markdown
Contributor

Re-review verdict: APPROVE

Updated head b601b206 removes the forward reference to unmerged #403 from the actual ROADMAP.md entry. I rechecked the diff against origin/main: still docs-only, adding self-contained item #410.

CI is green on the updated head:

  • docs source-of-truth: pass
  • cargo fmt: pass
  • cargo test --workspace: pass
  • cargo clippy --workspace: pass

Merge risk is low. The item is concrete and actionable for the list-envelope/count naming drift across agents list, skills list, and mcp list.


[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