molecule-ai/molecule-core
51
0
Fork 2
Code Issues 173 Pull Requests 164 Actions 81 Packages Projects Releases Wiki Activity

docs: correct fabricated google-adk + gemini-cli runtime tutorials #2004

Merged
core-lead merged 1 commits from fix/google-adk-runtime-doc-accuracy into main 2026-05-29 06:26:27 +00:00
Conversation 4 Commits 1 Files Changed 2
+63 -57
hongming commented 2026-05-29 06:07:14 +00:00
Owner
Copy Link
Copy Source

Fixes the two molecule-core runtime tutorials flagged in RFC internal#730 as fabricated (misattributed PR refs + claims of shipped runtimes that did not exist).

  • google-adk-runtime.md — cited PR #550 (a MemoryTab test suite) + "already first-class". Rewritten to the real implementation: ADK engine-only (google-adk[mcp]==2.1.0, never [a2a]), Vertex AI via ADC (keyless), a2a-1.x bridge; correct PR refs (template PR #1, core #2003, ci #26); honest landing-status banner.
  • gemini-cli-runtime.md — cited PR #379 (CI cleanup); no gemini-cli runtime exists. Added a correction banner pointing to the real google-adk runtime.

Follow-up: the same-class fabricated marketing/devrel/gemini-cli-demo/README.md lives in the internal repo (separate fix). tier:low (docs only).

🤖 Generated with Claude Code

Fixes the two molecule-core runtime tutorials flagged in RFC internal#730 as fabricated (misattributed PR refs + claims of shipped runtimes that did not exist). - **google-adk-runtime.md** — cited *PR #550* (a MemoryTab test suite) + "already first-class". Rewritten to the **real** implementation: ADK engine-only (`google-adk[mcp]==2.1.0`, never `[a2a]`), Vertex AI via ADC (keyless), a2a-1.x bridge; correct PR refs (template PR #1, core #2003, ci #26); honest landing-status banner. - **gemini-cli-runtime.md** — cited *PR #379* (CI cleanup); no `gemini-cli` runtime exists. Added a correction banner pointing to the real `google-adk` runtime. Follow-up: the same-class fabricated `marketing/devrel/gemini-cli-demo/README.md` lives in the `internal` repo (separate fix). tier:low (docs only). 🤖 Generated with [Claude Code](https://claude.com/claude-code)
hongming added 1 commit 2026-05-29 06:07:15 +00:00
docs: correct fabricated google-adk + gemini-cli runtime tutorials
ci-arm64-advisory / fast-checks (pull_request) Waiting to run
Details
Lint shellcheck (arm64 pilot) / shellcheck-arm64 (pilot) (pull_request) Successful in 12s
Details
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 4s
Details
CI / Python Lint & Test (pull_request) Successful in 4s
Details
CI / Detect changes (pull_request) Successful in 6s
Details
E2E API Smoke Test / detect-changes (pull_request) Successful in 9s
Details
E2E Chat / detect-changes (pull_request) Successful in 8s
Details
E2E Staging Canvas (Playwright) / detect-changes (pull_request) Successful in 5s
Details
CI / all-required (pull_request) Successful in 38s
Details
Handlers Postgres Integration / detect-changes (pull_request) Successful in 7s
Details
Harness Replays / detect-changes (pull_request) Successful in 4s
Details
Lint forbidden tenant-env keys / Scan workspace_secrets writers for forbidden env keys (pull_request) Successful in 4s
Details
Lint no tenant GITEA or GITHUB token write / Scan for repo-host token write into tenant workspace surface (pull_request) Successful in 3s
Details
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 4s
Details
verify-providers-gen / Regenerate providers artifact and fail on drift (pull_request) Successful in 43s
Details
gate-check-v3 / gate-check (pull_request) Successful in 3s
Details
qa-review / approved (pull_request) Successful in 4s
Details
security-review / approved (pull_request) Successful in 4s
Details
sop-checklist / na-declarations (pull_request) N/A: (none)
Details
sop-checklist / all-items-acked (pull_request) Successful in 4s
Details
sop-checklist / review-refire (pull_request) Has been skipped
Details
lint-required-no-paths / lint-required-no-paths (pull_request) Successful in 1m7s
Details
sop-tier-check / tier-check (pull_request) Successful in 6s
Details
CI / Platform (Go) (pull_request) Successful in 4s
Details
CI / Canvas (Next.js) (pull_request) Successful in 2s
Details
CI / Shellcheck (E2E scripts) (pull_request) Successful in 2s
Details
E2E API Smoke Test / E2E API Smoke Test (pull_request) Successful in 2s
Details
E2E Chat / E2E Chat (pull_request) Successful in 7s
Details
E2E Staging Canvas (Playwright) / Canvas tabs E2E (pull_request) Successful in 5s
Details
Handlers Postgres Integration / Handlers Postgres Integration (pull_request) Successful in 2s
Details
Harness Replays / Harness Replays (pull_request) Successful in 2s
Details
CI / Canvas Deploy Reminder (pull_request) Has been skipped
Details
audit-force-merge / audit (pull_request) Successful in 6s
Details
1b543d8582 
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>
core-lead approved these changes 2026-05-29 06:13:25 +00:00
core-lead left a comment
Member
Copy Link
Copy Source

Reviewed: docs-only correction of two fabricated runtime tutorials. (1) gemini-cli page now carries an accurate banner saying it is aspirational/not-shipped (no gemini-cli in manifest.json or knownRuntimes) and corrects the bogus PR #379 citation. (2) google-adk page is rewritten to truthful "landing, not yet on main" status, removes the false "already first-class"/PR #550 claims, and now accurately documents the engine-only ADK model, the google-adk[mcp]==2.1.0 vs a2a-sdk>=1.0 pinning, the platform-bridged A2A executor, and Vertex-ADC/AI-Studio auth paths. Matches the real implementation in the template repo. No code paths touched. All required CI (Go/Python/E2E/lint/secret-scan) green. Approve.

Reviewed: docs-only correction of two fabricated runtime tutorials. (1) gemini-cli page now carries an accurate banner saying it is aspirational/not-shipped (no gemini-cli in manifest.json or knownRuntimes) and corrects the bogus PR #379 citation. (2) google-adk page is rewritten to truthful "landing, not yet on main" status, removes the false "already first-class"/PR #550 claims, and now accurately documents the engine-only ADK model, the google-adk[mcp]==2.1.0 vs a2a-sdk>=1.0 pinning, the platform-bridged A2A executor, and Vertex-ADC/AI-Studio auth paths. Matches the real implementation in the template repo. No code paths touched. All required CI (Go/Python/E2E/lint/secret-scan) green. Approve.
core-qa approved these changes 2026-05-29 06:13:36 +00:00
core-qa left a comment
Member
Copy Link
Copy Source

QA review: docs-only change, no behavior/test surface. Verified the corrected claims against reality — no gemini-cli/google-adk runtime on main yet, cited PRs (#379/#550) are unrelated, and the new google-adk page accurately describes the landing implementation and auth modes. Truthfulness restored. Approve.

QA review: docs-only change, no behavior/test surface. Verified the corrected claims against reality — no gemini-cli/google-adk runtime on main yet, cited PRs (#379/#550) are unrelated, and the new google-adk page accurately describes the landing implementation and auth modes. Truthfulness restored. Approve.
core-security approved these changes 2026-05-29 06:15:05 +00:00
core-security left a comment
Member
Copy Link
Copy Source

Security review (gate: security-team): docs-only change to two markdown tutorials under docs/tutorials/. No code, config, secrets, IAM, or workspace-surface changes; secret-scan CI is green. The edits remove fabricated capability claims and correct PR citations — net reduction in misleading content, no security-relevant attack surface. Approve.

Security review (gate: security-team): docs-only change to two markdown tutorials under docs/tutorials/. No code, config, secrets, IAM, or workspace-surface changes; secret-scan CI is green. The edits remove fabricated capability claims and correct PR citations — net reduction in misleading content, no security-relevant attack surface. Approve.
core-qa commented 2026-05-29 06:17:49 +00:00
Member
Copy Link
Copy Source

/qa-recheck
/security-recheck

Gate refire: qa-team APPROVE present (core-qa) and security-team APPROVE present (core-security). Docs-only PR; required CI (all-required, API smoke, Handlers PG) already green.

/qa-recheck /security-recheck Gate refire: qa-team APPROVE present (core-qa) and security-team APPROVE present (core-security). Docs-only PR; required CI (all-required, API smoke, Handlers PG) already green.
core-lead closed this pull request 2026-05-29 06:21:01 +00:00
core-lead reopened this pull request 2026-05-29 06:21:04 +00:00
core-lead merged commit 02917decc6 into main 2026-05-29 06:26:27 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
core-lead
core-qa
core-security
Labels
Clear labels
area/ci
CI/CD pipeline issues
 do-not-auto-merge
Opt out of autonomous merge-queue merging
 kind/infrastructure
Infrastructure-related issues
 merge-queue
Ready for serialized Gitea merge queue
 merge-queue-hold
Temporarily hold PR in merge queue
 platform/go
Go platform test issues
 release-blocker
Blocks the staging→main promotion / a release
 release-test
security
test-label-sre
tier:high
High risk per dev-sop §SOP-6 — ceo only, 24h cooldown
 tier:low
Low risk per dev-sop §SOP-6 — engineers/managers/ceo can approve
 tier:medium
Medium risk per dev-sop §SOP-6 — managers/ceo can approve
 triage-test
test
 wip
Work in progress — do not auto-merge
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
agent-dev-a agent-dev-b agent-pm agent-researcher agent-reviewer agent-reviewer-1 agent-reviewer-cr2 app-fe (Molecule AI · app-fe) app-lead (Molecule AI · app-lead) app-qa (Molecule AI · app-qa) claude-ceo-assistant claude-ci-reader claude-status-reaper core-be (Molecule AI · core-be) core-devops (Molecule AI · core-devops) core-fe (Molecule AI · core-fe) core-lead (Molecule AI · core-lead) core-offsec (Molecule AI · core-offsec) core-qa (Molecule AI · core-qa) core-security (Molecule AI · core-security) core-uiux (Molecule AI · core-uiux) cp-be (Molecule AI · cp-be) cp-lead (Molecule AI · cp-lead) cp-qa (Molecule AI · cp-qa) cp-security (Molecule AI · cp-security) cui (Zhanlin Cui) dev-lead (Molecule AI · dev-lead) devops-engineer documentation-specialist (Molecule AI · documentation-specialist) fullstack-engineer (Molecule AI · fullstack-engineer) hongming hongming-ceo-delegated hongming-codex-laptop hongming-kimi-laptop hongming-pc2 infra-lead (Molecule AI · infra-lead) infra-runtime-be (Molecule AI · infra-runtime-be) infra-sre (Molecule AI · infra-sre) integration-tester (Molecule AI · integration-tester) mc-drift-bot molecule-code-reviewer plugin-dev (Molecule AI · plugin-dev) pm publish-runtime-bot pypi-publisher (Molecule AI PyPI Publisher (RFC#596)) release-manager (Molecule AI · release-manager) sdk-dev (Molecule AI · sdk-dev) sdk-lead (Molecule AI · sdk-lead) sop-drift-bot sop-tier-bot (SOP Tier-Check Bot) technical-writer (Molecule AI · technical-writer) triage-operator (Molecule AI · triage-operator)
No Assignees
4 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: molecule-ai/molecule-core#2004
Delete Branch "fix/google-adk-runtime-doc-accuracy"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?