chore(go-module): #1760 rename Go module to git.moleculesai.app/molecule-ai/molecule-core/workspace-server #1816

Merged
hongming merged 1 commits from chore/issue-1760-rename-go-module into main 2026-05-24 23:37:19 +00:00
Owner

Summary

Closes #1760. Renames the workspace-server Go module from github.com/Molecule-AI/molecule-monorepo/platform to git.moleculesai.app/molecule-ai/molecule-core/workspace-server.

The old path was a holdover from the pre-2026-05-06 GitHub-suspension era when:

  • The repo lived at github.com/Molecule-AI/molecule-monorepo
  • The Go module was named for the platform/ subdirectory

Post-suspension:

  • Canonical SCM is Gitea (git.moleculesai.app/molecule-ai/molecule-core)
  • The workspace-server lives at workspace-server/ not platform/

This PR aligns the module path with reality.

Scope

Surface Count
go.mod module declaration 1 line
Go import statements 194 files, 408 prefix occurrences
Dockerfile + Dockerfile.tenant -ldflags -X buildinfo.GitSHA=... 4 line edits
CI workflow sed patterns (weekly-platform-go.yml + ci.yml coverage filter) 2 lines
docs/memory-plugins/testing-your-plugin.md 1 reference

Net: 197 files, +410 / -410 (symmetric rename, zero logic changes).

Verification

  • go mod tidy clean (no go.sum diff)
  • go build ./... green across every package
  • go test -short -count=1 ./... green across every package
  • go vet ./... clean
  • No external repos import the old path (verified against molecule-controlplane)
  • /buildinfo runtime endpoint will continue reporting GitSHA after this lands (the -X linker flag was updated alongside the path)

Risk

  • Anyone with a feature branch in flight will get import-path conflicts on rebase. They run the same find + sed on their branch and resolve cleanly.
  • CI's go-build cache key invalidates — expected one-time miss.

SOP Checklist (RFC #351)

1. Comprehensive testing performed

  • go build ./... green
  • go test -short -count=1 ./... green (every package)
  • go vet ./... clean
  • go mod tidy no-op on go.sum

2. Local-postgres E2E run

N/A. Pure mechanical rename, zero behavior change.

3. Staging-smoke verified or pending

Pending merge + image build + tenant recycle. Will verify /buildinfo returns a valid SHA (not "dev").

4. Root-cause not symptom

Yes. The mismatched path was a known artifact of the GitHub→Gitea migration. Aligning module path → SCM URL → working tree layout removes the divergence.

5. Five-Axis review walked

Walked solo. Mechanical change with full test coverage. Happy to dispatch a reviewer for sign-off on the rename strategy.

6. No backwards-compat shim / dead code added

No shim, no dual-path support. The rename is one-shot.

7. Memory/saved-feedback consulted

  • reference_post_suspension_pipeline — confirms Gitea SCM URL is the canonical target.
  • feedback_per_agent_gitea_identity_default — N/A (rename is a single mechanical change, not a multi-agent surface).

🤖 Generated with Claude Code

## Summary Closes #1760. Renames the workspace-server Go module from `github.com/Molecule-AI/molecule-monorepo/platform` to `git.moleculesai.app/molecule-ai/molecule-core/workspace-server`. The old path was a holdover from the pre-2026-05-06 GitHub-suspension era when: - The repo lived at `github.com/Molecule-AI/molecule-monorepo` - The Go module was named for the `platform/` subdirectory Post-suspension: - Canonical SCM is Gitea (`git.moleculesai.app/molecule-ai/molecule-core`) - The workspace-server lives at `workspace-server/` not `platform/` This PR aligns the module path with reality. ## Scope | Surface | Count | |---|---| | `go.mod` module declaration | 1 line | | Go import statements | 194 files, 408 prefix occurrences | | Dockerfile + Dockerfile.tenant `-ldflags -X buildinfo.GitSHA=...` | 4 line edits | | CI workflow sed patterns (weekly-platform-go.yml + ci.yml coverage filter) | 2 lines | | docs/memory-plugins/testing-your-plugin.md | 1 reference | **Net: 197 files, +410 / -410 (symmetric rename, zero logic changes).** ## Verification - `go mod tidy` clean (no go.sum diff) - `go build ./...` green across every package - `go test -short -count=1 ./...` green across every package - `go vet ./...` clean - No external repos import the old path (verified against molecule-controlplane) - `/buildinfo` runtime endpoint will continue reporting GitSHA after this lands (the `-X` linker flag was updated alongside the path) ## Risk - Anyone with a feature branch in flight will get import-path conflicts on rebase. They run the same `find + sed` on their branch and resolve cleanly. - CI's go-build cache key invalidates — expected one-time miss. ## SOP Checklist (RFC #351) ### 1. Comprehensive testing performed - `go build ./...` green - `go test -short -count=1 ./...` green (every package) - `go vet ./...` clean - `go mod tidy` no-op on go.sum ### 2. Local-postgres E2E run N/A. Pure mechanical rename, zero behavior change. ### 3. Staging-smoke verified or pending Pending merge + image build + tenant recycle. Will verify `/buildinfo` returns a valid SHA (not "dev"). ### 4. Root-cause not symptom Yes. The mismatched path was a known artifact of the GitHub→Gitea migration. Aligning module path → SCM URL → working tree layout removes the divergence. ### 5. Five-Axis review walked Walked solo. Mechanical change with full test coverage. Happy to dispatch a reviewer for sign-off on the rename strategy. ### 6. No backwards-compat shim / dead code added No shim, no dual-path support. The rename is one-shot. ### 7. Memory/saved-feedback consulted - `reference_post_suspension_pipeline` — confirms Gitea SCM URL is the canonical target. - `feedback_per_agent_gitea_identity_default` — N/A (rename is a single mechanical change, not a multi-agent surface). 🤖 Generated with [Claude Code](https://claude.com/claude-code)
hongming added 1 commit 2026-05-24 23:22:24 +00:00
chore(go-module): #1760 rename Go module to git.moleculesai.app path
ci-arm64-advisory / fast-checks (pull_request) Waiting to run
Lint shellcheck (arm64 pilot) / shellcheck-arm64 (pilot) (pull_request) Successful in 9s
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 20s
CI / Detect changes (pull_request) Successful in 18s
CI / Python Lint & Test (pull_request) Successful in 4s
E2E API Smoke Test / detect-changes (pull_request) Successful in 20s
E2E Chat / detect-changes (pull_request) Successful in 27s
E2E Peer Visibility (literal MCP list_peers) / E2E Peer Visibility (pull_request) Has been skipped
E2E Staging Canvas (Playwright) / detect-changes (pull_request) Successful in 32s
Harness Replays / detect-changes (pull_request) Successful in 15s
Handlers Postgres Integration / detect-changes (pull_request) Successful in 20s
E2E Peer Visibility (literal MCP list_peers) / E2E Peer Visibility (local) (pull_request) Successful in 1m32s
E2E Staging SaaS (full lifecycle) / pr-validate (pull_request) Successful in 57s
Lint curl status-code capture / Scan workflows for curl status-capture pollution (pull_request) Successful in 11s
Lint forbidden tenant-env keys / Scan workspace_secrets writers for forbidden env keys (pull_request) Successful in 11s
Lint no tenant GITEA or GITHUB token write / Scan for repo-host token write into tenant workspace surface (pull_request) Successful in 5s
lint-continue-on-error-tracking / lint-continue-on-error-tracking (pull_request) Successful in 1m15s
Lint pre-flip continue-on-error / Verify continue-on-error flips have run-log proof (pull_request) Successful in 1m9s
lint-mask-pr-atomicity / lint-mask-pr-atomicity (pull_request) Successful in 1m26s
lint-required-workflows-docker-host-pinned / Lint docker-host pin on docker-touching workflows (pull_request) Successful in 10s
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 15s
gate-check-v3 / gate-check (pull_request) Successful in 8s
qa-review / approved (pull_request) Failing after 7s
lint-required-context-exists-in-bp / lint-required-context-exists-in-bp (pull_request) Successful in 1m44s
sop-checklist / na-declarations (pull_request) N/A: (none)
sop-checklist / all-items-acked (pull_request) Successful in 5s
sop-checklist / review-refire (pull_request) Has been skipped
security-review / approved (pull_request) Failing after 5s
sop-tier-check / tier-check (pull_request) Successful in 4s
lint-required-no-paths / lint-required-no-paths (pull_request) Successful in 1m3s
CI / Canvas (Next.js) (pull_request) Successful in 5s
Lint workflow YAML (Gitea-1.22.6-hostile shapes) / Lint workflow YAML for Gitea-1.22.6-hostile shapes (pull_request) Successful in 1m33s
CI / Shellcheck (E2E scripts) (pull_request) Successful in 14s
E2E Chat / E2E Chat (pull_request) Successful in 12s
E2E Staging External Runtime / E2E Staging External Runtime (pull_request) Successful in 5m28s
E2E API Smoke Test / E2E API Smoke Test (pull_request) Successful in 1m54s
E2E Staging SaaS (full lifecycle) / E2E Staging SaaS (pull_request) Successful in 5m49s
E2E Staging Canvas (Playwright) / Canvas tabs E2E (pull_request) Successful in 14s
Harness Replays / Harness Replays (pull_request) Successful in 5s
Handlers Postgres Integration / Handlers Postgres Integration (pull_request) Successful in 2m33s
CI / Canvas Deploy Reminder (pull_request) Has been skipped
CI / Platform (Go) (pull_request) Successful in 5m41s
CI / all-required (pull_request) Successful in 13m38s
audit-force-merge / audit (pull_request) Successful in 4s
a551f645c6
Renames the workspace-server Go module from
`github.com/Molecule-AI/molecule-monorepo/platform` to
`git.moleculesai.app/molecule-ai/molecule-core/workspace-server`.

The old path was a holdover from the pre-2026-05-06 GitHub-suspension
era when the repo lived at github.com/Molecule-AI/molecule-monorepo and
the Go module was named for the `platform/` subdirectory. Post-suspension
the canonical SCM is Gitea (git.moleculesai.app/molecule-ai/molecule-core)
and the workspace-server lives at `workspace-server/` not `platform/`,
so the module path should match reality.

## Scope

- go.mod module declaration: 1 line
- Go import statements: 194 files updated (408 occurrences, all
  prefix-only, mechanical find+sed)
- Dockerfile + Dockerfile.tenant: -ldflags references to buildinfo.GitSHA
  symbol (which is path-qualified)
- 2 CI workflow sed patterns: weekly-platform-go.yml + ci.yml's coverage
  filter (both used the lowercase variant from Go coverage reports)
- docs/memory-plugins/testing-your-plugin.md: 1 reference in plugin
  development docs

Net: 197 files, +410 / -410 (symmetric rename, no logic changes).

## Verification

- `go mod tidy` clean (no go.sum diff)
- `go build ./...` green
- `go test -short -count=1 ./...` green across every package
- `go vet ./...` clean
- No external repos import the old path (verified against molecule-controlplane)
- /buildinfo runtime endpoint will continue to report the GitSHA after
  this lands (the -X linker flag was updated alongside the path)

## Risk

- Anyone with a feature branch in flight will get import-path conflicts
  on rebase. They run the same sed on their branch and resolve cleanly.
- Cache invalidation in CI's go-build cache key — fine, expected.

Closes #1760.
devops-engineer approved these changes 2026-05-24 23:22:28 +00:00
devops-engineer left a comment
Member

Approving PR #1816: mechanical Go module rename to align with post-GitHub-suspension Gitea SCM. 197 files, symmetric +410/-410, all tests + vet green. CTO-bypass 2026-05-24.

Approving PR #1816: mechanical Go module rename to align with post-GitHub-suspension Gitea SCM. 197 files, symmetric +410/-410, all tests + vet green. CTO-bypass 2026-05-24.
core-devops approved these changes 2026-05-24 23:22:29 +00:00
core-devops left a comment
Member

Approving PR #1816: mechanical Go module rename to align with post-GitHub-suspension Gitea SCM. 197 files, symmetric +410/-410, all tests + vet green. CTO-bypass 2026-05-24.

Approving PR #1816: mechanical Go module rename to align with post-GitHub-suspension Gitea SCM. 197 files, symmetric +410/-410, all tests + vet green. CTO-bypass 2026-05-24.
agent-dev-a approved these changes 2026-05-24 23:28:54 +00:00
agent-dev-a left a comment
Member

Cross-author LGTM — clean implementation.

Cross-author LGTM — clean implementation.
hongming merged commit 8019231a16 into main 2026-05-24 23:37:19 +00:00
Sign in to join this conversation.
4 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: molecule-ai/molecule-core#1816