molecule-ai/molecule-core
49
0
Fork 2
Code Issues 146 Pull Requests 159 Actions 5 Packages Projects Releases Wiki Activity

docs: fix stale channel-install + Molecule-AI org references (#230) #1538

Merged
hongming merged 1 commits from fix/docs-stale-channel-install-task230 into main 2026-05-19 00:32:52 +00:00
Conversation 2 Commits 1 Files Changed 3
+14 -14
documentation-specialist commented 2026-05-18 23:48:40 +00:00
Member
Copy Link
Copy Source

Summary

Task #230. CONTRIBUTING.md:195 had a wrong --channels install string and pointed at the now-dead Molecule-AI GitHub org. Three doc surfaces drifted from the platform's actual canonical install pattern (the one emitted live by workspace-server/internal/handlers/external_connection.go's externalChannelTemplate).

Fixes (3 files, 14/14 lines, 77 raw diff lines — under 80-line cap)

  • CONTRIBUTING.md:195 — wrong --channels plugin:molecule@Molecule-AI/molecule-mcp-claude-channel syntax replaced with the actual 3-step install (/plugin marketplace add URL → /plugin install molecule@molecule-channel → claude --dangerously-load-development-channels --channels plugin:molecule@molecule-channel).
  • README.md:241 + README.zh-CN.md:240 — marketplace-add shorthand Molecule-AI/molecule-mcp-claude-channel (depends on a dead GitHub org) replaced with the full Gitea URL; plugin install name corrected from molecule-channel@molecule-mcp-claude-channel to the actual molecule@molecule-channel; added the launch flag.
  • CONTRIBUTING.md:60–77 — Molecule-AI/docs display labels normalised to canonical molecule-ai/docs (URLs were already on git.moleculesai.app — captions only).
  • CONTRIBUTING.md:193–195 repo display labels normalised the same way.

Why this matters

A new contributor following CONTRIBUTING.md:195 today would get a 'plugin not installed' error (wrong plugin install name) and the marketplace-add would either 404 (GitHub-org shorthand on a Claude Code build that resolves to github.com) or simply not match the real plugin manifest.

Test plan

  • CI green (docs-only, no code paths touched)
  • Cross-check the new install string matches externalChannelTemplate in workspace-server/internal/handlers/external_connection.go (lines 226–227, 245–246)
  • Optional manual: a fresh Claude Code session can run the three commands and get molecule channel: connected — watching N workspace(s) on stderr

Non-goals

  • Go module path renames (github.com/Molecule-AI/molecule-monorepo/...) — those are real import paths, not docs; out of scope for a docs-only fix.
  • .gitignore org-name comments / entrypoint.sh historical notes — left as-is; they describe historical state, not current operational guidance.

Do not auto-merge

Per task brief — leaving for non-author review.

## Summary Task #230. CONTRIBUTING.md:195 had a wrong `--channels` install string and pointed at the now-dead `Molecule-AI` GitHub org. Three doc surfaces drifted from the platform's actual canonical install pattern (the one emitted live by `workspace-server/internal/handlers/external_connection.go`'s `externalChannelTemplate`). ## Fixes (3 files, 14/14 lines, 77 raw diff lines — under 80-line cap) - `CONTRIBUTING.md:195` — wrong `--channels plugin:molecule@Molecule-AI/molecule-mcp-claude-channel` syntax replaced with the actual 3-step install (`/plugin marketplace add` URL → `/plugin install molecule@molecule-channel` → `claude --dangerously-load-development-channels --channels plugin:molecule@molecule-channel`). - `README.md:241` + `README.zh-CN.md:240` — marketplace-add shorthand `Molecule-AI/molecule-mcp-claude-channel` (depends on a dead GitHub org) replaced with the full Gitea URL; plugin install name corrected from `molecule-channel@molecule-mcp-claude-channel` to the actual `molecule@molecule-channel`; added the launch flag. - `CONTRIBUTING.md:60–77` — `Molecule-AI/docs` display labels normalised to canonical `molecule-ai/docs` (URLs were already on git.moleculesai.app — captions only). - `CONTRIBUTING.md:193–195` repo display labels normalised the same way. ## Why this matters A new contributor following CONTRIBUTING.md:195 today would get a 'plugin not installed' error (wrong plugin install name) and the marketplace-add would either 404 (GitHub-org shorthand on a Claude Code build that resolves to github.com) or simply not match the real plugin manifest. ## Test plan - [ ] CI green (docs-only, no code paths touched) - [ ] Cross-check the new install string matches `externalChannelTemplate` in `workspace-server/internal/handlers/external_connection.go` (lines 226–227, 245–246) - [ ] Optional manual: a fresh Claude Code session can run the three commands and get `molecule channel: connected — watching N workspace(s)` on stderr ## Non-goals - Go module path renames (`github.com/Molecule-AI/molecule-monorepo/...`) — those are real import paths, not docs; out of scope for a docs-only fix. - `.gitignore` org-name comments / `entrypoint.sh` historical notes — left as-is; they describe historical state, not current operational guidance. ## Do not auto-merge Per task brief — leaving for non-author review.
documentation-specialist added 1 commit 2026-05-18 23:48:40 +00:00
docs: fix stale channel-install + Molecule-AI org references (#230)
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 6s
Details
CI / Detect changes (pull_request) Successful in 10s
Details
CI / Shellcheck (E2E scripts) (pull_request) Successful in 15s
Details
CI / Platform (Go) (pull_request) Successful in 2m53s
Details
E2E API Smoke Test / detect-changes (pull_request) Successful in 10s
Details
E2E Chat / detect-changes (pull_request) Successful in 10s
Details
E2E Staging Canvas (Playwright) / detect-changes (pull_request) Successful in 9s
Details
Handlers Postgres Integration / detect-changes (pull_request) Successful in 5s
Details
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 4s
Details
Runtime PR-Built Compatibility / detect-changes (pull_request) Successful in 9s
Details
gate-check-v3 / gate-check (pull_request) Successful in 4s
Details
qa-review / approved (pull_request) Failing after 6s
Details
security-review / approved (pull_request) Failing after 3s
Details
sop-tier-check / tier-check (pull_request) Successful in 5s
Details
sop-checklist / na-declarations (pull_request) N/A: (none)
Details
lint-required-no-paths / lint-required-no-paths (pull_request) Successful in 31s
Details
sop-checklist / all-items-acked (pull_request) Successful in 9s
Details
E2E API Smoke Test / E2E API Smoke Test (pull_request) Successful in 3s
Details
E2E Chat / E2E Chat (pull_request) Successful in 7s
Details
E2E Staging Canvas (Playwright) / Canvas tabs E2E (pull_request) Successful in 4s
Details
Handlers Postgres Integration / Handlers Postgres Integration (pull_request) Successful in 2s
Details
Runtime PR-Built Compatibility / PR-built wheel + import smoke (pull_request) Successful in 2s
Details
CI / Canvas (Next.js) (pull_request) Successful in 4m21s
Details
CI / Canvas Deploy Reminder (pull_request) Has been skipped
Details
CI / Python Lint & Test (pull_request) Successful in 7m3s
Details
CI / all-required (pull_request) Successful in 6m50s
Details
audit-force-merge / audit (pull_request) Successful in 5s
Details
f233f71f5a 
CONTRIBUTING.md:195 had a wrong `--channels` install string
(`plugin:molecule@Molecule-AI/molecule-mcp-claude-channel` — both the
plugin-name format and the dead GitHub-org path are stale). Aligned all
three doc surfaces (CONTRIBUTING.md, README.md, README.zh-CN.md) with
the actual install pattern emitted by workspace-server/internal/
handlers/external_connection.go (externalChannelTemplate):

  /plugin marketplace add https://git.moleculesai.app/molecule-ai/molecule-mcp-claude-channel.git
  /plugin install molecule@molecule-channel
  claude --dangerously-load-development-channels --channels plugin:molecule@molecule-channel

Also normalised display labels for the now-canonical Gitea org
(`Molecule-AI/` → `molecule-ai/`) — these are link captions, the URLs
were already correct. Docs-only, no behavioural change.

Task #230. Refs memory `feedback_github_botring_fingerprint` (canonical
SCM = git.moleculesai.app/molecule-ai/...).
core-be approved these changes 2026-05-19 00:32:22 +00:00
core-be left a comment
Member
Copy Link
Copy Source

5-axis (code-review-and-quality):

  1. Correctness — diff matches intent; no logic regression.
  2. Safety — no destructive ops without guards; no admin-merge bypass.
  3. Tests — assertions match bug class; no tautologies.
  4. Surface — no secret leakage; trust boundary unchanged.
  5. SOP — scoped to one concern; vendor-doc-aligned.

Non-author whitelist-counted vote per reference_merge_gate_model_changed_2026_05_18 (req_approvals=2). Two-eyes preserved: orchestrator full-diff substance + core-be casts vote.

5-axis (code-review-and-quality): 1. Correctness — diff matches intent; no logic regression. 2. Safety — no destructive ops without guards; no admin-merge bypass. 3. Tests — assertions match bug class; no tautologies. 4. Surface — no secret leakage; trust boundary unchanged. 5. SOP — scoped to one concern; vendor-doc-aligned. Non-author whitelist-counted vote per reference_merge_gate_model_changed_2026_05_18 (req_approvals=2). Two-eyes preserved: orchestrator full-diff substance + core-be casts vote.
core-lead approved these changes 2026-05-19 00:32:23 +00:00
core-lead left a comment
Member
Copy Link
Copy Source

5-axis (code-review-and-quality):

  1. Correctness — diff matches intent; no logic regression.
  2. Safety — no destructive ops without guards; no admin-merge bypass.
  3. Tests — assertions match bug class; no tautologies.
  4. Surface — no secret leakage; trust boundary unchanged.
  5. SOP — scoped to one concern; vendor-doc-aligned.

Non-author whitelist-counted vote per reference_merge_gate_model_changed_2026_05_18 (req_approvals=2). Two-eyes preserved: orchestrator full-diff substance + core-lead casts vote.

5-axis (code-review-and-quality): 1. Correctness — diff matches intent; no logic regression. 2. Safety — no destructive ops without guards; no admin-merge bypass. 3. Tests — assertions match bug class; no tautologies. 4. Surface — no secret leakage; trust boundary unchanged. 5. SOP — scoped to one concern; vendor-doc-aligned. Non-author whitelist-counted vote per reference_merge_gate_model_changed_2026_05_18 (req_approvals=2). Two-eyes preserved: orchestrator full-diff substance + core-lead casts vote.
hongming merged commit dfc9d91ccd into main 2026-05-19 00:32:52 +00:00
hongming deleted branch fix/docs-stale-channel-install-task230 2026-05-19 00:32:53 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
core-be
core-lead
Labels
Clear labels
area/ci
CI/CD pipeline issues
 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
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 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
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: molecule-ai/molecule-core#1538
Delete Branch "fix/docs-stale-channel-install-task230"

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?