Skip to content

fix(list): cover all resource types and mask env secrets - #255

Merged
jeff-r2026 merged 1 commit into
mainfrom
fix/list-include-mcp
Jul 28, 2026
Merged

fix(list): cover all resource types and mask env secrets#255
jeff-r2026 merged 1 commit into
mainfrom
fix/list-include-mcp

Conversation

@jeff-r2026

Copy link
Copy Markdown
Collaborator

Summary

Audited resource-facing commands after MCP landed as the 7th type. Gaps:

Command Before After
teamai list Only skills/rules/docs/env; env values in plaintext All 7 types; env masked (--reveal to show)
teamai status Counts missed agents + mcp Counts every RESOURCE_TYPES entry
teamai remove help Said skills|rules (code already had agents) skills|rules|agents|mcp
teamai mcp list / hooks list Already complete unchanged (detailed views)

pull already reconciled mcp/hooks outside the main loop — no change needed. push correctly omits mcp/hooks (edit yaml in-repo).

Test plan

  • Unit: default list shows mcp/agents/hooks; env masked; --reveal shows plaintext; status counts agents/hooks/mcp
  • npx vitest run src/__tests__/status-list.test.ts src/__tests__/env-commands.test.ts
  • Built CLI: list --help lists all types; list mcp --source repo prints servers

Made with Cursor

teamai list defaulted to skills/rules/docs/env, so agents, hooks, and mcp
never appeared unless you already knew the type name. It also printed env
values in plaintext, unlike teamai env list.

Default list now walks RESOURCE_TYPES; mcp/hooks sections show useful
detail; env is masked unless --reveal. status counts agents and mcp too.
Help and remove accept mcp alongside skills/rules/agents.
@jeff-r2026
jeff-r2026 merged commit 9c885ae into main Jul 28, 2026
6 checks passed
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