molecule-ai/molecule-core
52
0
Fork 2
Code Issues 179 Pull Requests 58 Actions 5 Packages Projects Releases Wiki Activity

Rename Go module: github.com/Molecule-AI/molecule-monorepo/platform → git.moleculesai.app/molecule-ai/molecule-core/workspace-server #1760

New Issue
Closed
opened 2026-05-24 01:28:44 +00:00 by hongming · 0 comments
hongming commented 2026-05-24 01:28:44 +00:00
Owner
Copy Link
Copy Source

Rename Go module: github.com/Molecule-AI/molecule-monorepo/platform → git.moleculesai.app/molecule-ai/molecule-core/workspace-server

Problem

The Go module path in workspace-server/go.mod is:

module github.com/Molecule-AI/molecule-monorepo/platform

Two issues with this:

  1. github.com/Molecule-AI/... points at a host the org no longer uses. The GitHub org was suspended 2026-05-06; canonical SCM is now git.moleculesai.app/molecule-ai/<repo>. Anything that resolves Go module URLs against that path will 404.

  2. molecule-monorepo/platform doesn't match reality. The repo split into per-component shapes; this codebase lives in molecule-ai/molecule-core and the directory is workspace-server/, not platform/. New contributors hit immediate confusion (cd platform ⇒ no such directory).

canvas/ has no Go module of its own (TypeScript), so this is workspace-server-only — but 194 Go files import paths under the bad prefix.

Target

Per CTO 2026-05-23: git.moleculesai.app/molecule-ai/molecule-core/workspace-server

  • Matches the git clone URL exactly.
  • Matches the on-disk directory name (workspace-server).
  • Removes any reference to the suspended GitHub host.

Scope

$ grep -rl "github.com/Molecule-AI/molecule-monorepo" workspace-server/ | wc -l
194

Every internal import (internal/handlers, internal/memory, internal/provisioner, etc.) needs the prefix rewritten. Plus:

  • workspace-server/go.mod — module declaration line.
  • workspace-server/go.sum — unchanged (sum hashes are unaffected by the module's own name).
  • Any tooling (Dockerfile, scripts, CI workflows) referencing the path string verbatim.

Mechanical recipe

Once the queued PRs (#1737, #1742, #1747, #1749, #1753, #1755) have merged:

cd ~/molecule-core/workspace-server
OLD="github.com/Molecule-AI/molecule-monorepo/platform"
NEW="git.moleculesai.app/molecule-ai/molecule-core/workspace-server"

# 1. go.mod
sed -i.bak "s|^module $OLD|module $NEW|" go.mod && rm go.mod.bak

# 2. Every Go file (internal + cmd + pkg + tests)
grep -rl "$OLD" --include="*.go" . | xargs sed -i.bak "s|$OLD|$NEW|g"
find . -name "*.go.bak" -delete

# 3. Verify
go vet ./...
go test -short -count=1 ./...

Plus check for the path in non-Go files:

grep -rn "$OLD" . --include="*.Dockerfile" --include="*.yml" --include="*.yaml" --include="*.sh"

Cross-repo impact (must verify before merging)

If any of these repos imports paths under the old prefix, this rename breaks them:

  • molecule-controlplane — already split, no shared Go module.
  • molecule-mcp-server — TypeScript, no Go imports.
  • molecule-ai-workspace-runtime — Python, no Go imports.
  • molecule-mcp-claude-channel — verify.

Run git grep on each before flipping.

Sequencing

  1. Wait for #1737 + #1742 + #1747 + #1749 + #1753 + #1755 to merge. Rebasing 194 import edits over an unmerged queue creates avoidable churn — at least one PR per branch would need a force-rebase.
  2. Cross-repo check above.
  3. Single mechanical PR:
    • go.mod edit.
    • 194-file sed import rewrite.
    • go vet + go test -short -count=1 green.
    • Update any cross-references in Dockerfile.tenant, cmd/*/main.go build tags, etc.
  4. Post-merge: confirm CI cache is invalidated and the new module path resolves end-to-end on a fresh go mod download.

Tier

area:infra tier:high — touches every Go file in workspace-server but is mechanical. The risk is a forgotten cross-repo dependency, not a logic bug.

Refs

  • CTO 2026-05-23 decision (Claude Code session)
  • post-suspension SCM policy (~/.molecule-ai/AGENTS.md, feedback_post_suspension_pipeline)
  • GitHub org suspension 2026-05-06
# Rename Go module: `github.com/Molecule-AI/molecule-monorepo/platform` → `git.moleculesai.app/molecule-ai/molecule-core/workspace-server` ## Problem The Go module path in `workspace-server/go.mod` is: ``` module github.com/Molecule-AI/molecule-monorepo/platform ``` Two issues with this: 1. **`github.com/Molecule-AI/...`** points at a host the org no longer uses. The GitHub org was suspended 2026-05-06; canonical SCM is now `git.moleculesai.app/molecule-ai/<repo>`. Anything that resolves Go module URLs against that path will 404. 2. **`molecule-monorepo/platform`** doesn't match reality. The repo split into per-component shapes; this codebase lives in `molecule-ai/molecule-core` and the directory is `workspace-server/`, not `platform/`. New contributors hit immediate confusion (`cd platform` ⇒ no such directory). `canvas/` has no Go module of its own (TypeScript), so this is workspace-server-only — but **194 Go files** import paths under the bad prefix. ## Target Per CTO 2026-05-23: **`git.moleculesai.app/molecule-ai/molecule-core/workspace-server`** - Matches the `git clone` URL exactly. - Matches the on-disk directory name (`workspace-server`). - Removes any reference to the suspended GitHub host. ## Scope ``` $ grep -rl "github.com/Molecule-AI/molecule-monorepo" workspace-server/ | wc -l 194 ``` Every internal import (`internal/handlers`, `internal/memory`, `internal/provisioner`, etc.) needs the prefix rewritten. Plus: - `workspace-server/go.mod` — module declaration line. - `workspace-server/go.sum` — unchanged (sum hashes are unaffected by the module's own name). - Any tooling (Dockerfile, scripts, CI workflows) referencing the path string verbatim. ## Mechanical recipe Once the queued PRs (#1737, #1742, #1747, #1749, #1753, #1755) have merged: ```bash cd ~/molecule-core/workspace-server OLD="github.com/Molecule-AI/molecule-monorepo/platform" NEW="git.moleculesai.app/molecule-ai/molecule-core/workspace-server" # 1. go.mod sed -i.bak "s|^module $OLD|module $NEW|" go.mod && rm go.mod.bak # 2. Every Go file (internal + cmd + pkg + tests) grep -rl "$OLD" --include="*.go" . | xargs sed -i.bak "s|$OLD|$NEW|g" find . -name "*.go.bak" -delete # 3. Verify go vet ./... go test -short -count=1 ./... ``` Plus check for the path in non-Go files: ```bash grep -rn "$OLD" . --include="*.Dockerfile" --include="*.yml" --include="*.yaml" --include="*.sh" ``` ## Cross-repo impact (must verify before merging) If any of these repos imports paths under the old prefix, this rename breaks them: - `molecule-controlplane` — already split, no shared Go module. - `molecule-mcp-server` — TypeScript, no Go imports. - `molecule-ai-workspace-runtime` — Python, no Go imports. - `molecule-mcp-claude-channel` — verify. Run `git grep` on each before flipping. ## Sequencing 1. **Wait for #1737 + #1742 + #1747 + #1749 + #1753 + #1755** to merge. Rebasing 194 import edits over an unmerged queue creates avoidable churn — at least one PR per branch would need a force-rebase. 2. **Cross-repo check** above. 3. **Single mechanical PR**: - `go.mod` edit. - 194-file `sed` import rewrite. - `go vet` + `go test -short -count=1` green. - Update any cross-references in `Dockerfile.tenant`, `cmd/*/main.go` build tags, etc. 4. **Post-merge**: confirm CI cache is invalidated and the new module path resolves end-to-end on a fresh `go mod download`. ## Tier `area:infra` `tier:high` — touches every Go file in workspace-server but is mechanical. The risk is a forgotten cross-repo dependency, not a logic bug. ## Refs - CTO 2026-05-23 decision (Claude Code session) - post-suspension SCM policy (`~/.molecule-ai/AGENTS.md`, `feedback_post_suspension_pipeline`) - GitHub org suspension 2026-05-06
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
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) godwin 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
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: molecule-ai/molecule-core#1760
Delete Branch "%!s()"

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?