Skip to content

ci(gh-wrapper): translate --assignee @me → --label team:<role> (fixes #1957) - #2003

Merged
HongmingWang-Rabbit merged 1 commit into
stagingfrom
ci/gh-wrapper-identity-shim
Apr 24, 2026
Merged

ci(gh-wrapper): translate --assignee @me → --label team:<role> (fixes #1957)#2003
HongmingWang-Rabbit merged 1 commit into
stagingfrom
ci/gh-wrapper-identity-shim

Conversation

@HongmingWang-Rabbit

Copy link
Copy Markdown
Contributor

[Molecule-Platform-Evolvement-Manager]

Fixes #1957. Post #1933 retro item #5.

Why

All agents share one PAT, so gh issue create --assignee @me resolves to the CEO. Today's "6 issues @me for 7 cycles" defect signal turned out to be CEO-load misclassified as team-stagnation.

What it does

Two new cases in the existing arg-walk loop:

  • --assignee @me--label team:<role-slug> (role from GIT_AUTHOR_NAME, lowercased + dashed)
  • --reviewer @me → dropped (review-bot scans labels, not requests)
  • Real user names pass through unchanged

Acceptance

After this lands and templates pull the updated wrapper:

  • gh issue create --title foo --assignee @me from Core-BE → opens with label team:core-be and no assignee
  • gh issue list --label team:core-be becomes the discovery query for backend work
  • The maintenance "issues @me" metric becomes meaningful again (CEO actually has CEO work)

Agent prompts don't need updates — the wrapper is transparent.

Fixes #1957. All agents share one PAT, so `gh issue create --assignee @me`
resolves to the CEO. Today's "6 issues @me for 7 cycles" defect signal
turned out to be CEO-load misclassified as team-stagnation.

Translation rules:
- `--assignee @me` → `--label team:<role-slug>`
- `--reviewer @me` → dropped (review-bot scans labels, not requests)
- `--assignee user` (real user) → unchanged

role-slug derived from GIT_AUTHOR_NAME ("Molecule AI Core-BE" → "core-be").
The wrapper already handled the title-prefix + body-footer transforms;
these are just two more cases in the existing arg-walk loop.

Backward compat: any agent prompt that doesn't use @me passes through
unchanged. Agents don't need prompt updates — the wrapper is transparent.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@HongmingWang-Rabbit
HongmingWang-Rabbit added this pull request to the merge queue Apr 24, 2026
Merged via the queue into staging with commit 92ce37a Apr 24, 2026
13 checks passed
@molecule-ai
molecule-ai Bot deleted the ci/gh-wrapper-identity-shim branch May 20, 2026 06:21
HongmingWang-Rabbit pushed a commit that referenced this pull request Jun 12, 2026
Both tutorials cited misattributed PRs and claimed shipped runtimes that
didn't exist (RFC internal#730 finding):
- google-adk-runtime.md: cited 'PR #550' (actually a MemoryTab test suite) +
  'already first-class'. Rewritten to the REAL implementation — ADK engine-only
  (google-adk[mcp]==2.1.0, no [a2a]), Vertex AI via ADC (keyless), a2a-1.x
  bridge — with correct PR refs (template PR #1, core #2003, ci #26) + a
  landing-status banner.
- gemini-cli-runtime.md: cited 'PR #379' (actually CI cleanup); no gemini-cli
  runtime exists in manifest/knownRuntimes. Added a correction banner pointing
  to the real google-adk runtime.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
HongmingWang-Rabbit pushed a commit that referenced this pull request Jun 12, 2026
…wnRuntimes + canvas)' (#2003) from feat/register-google-adk-runtime into main
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.

[platform/identity] Agent gh CLI calls collapse identity to Hongming via shared PAT — issues + comments + PRs all assigned to CEO

1 participant