Skip to content

feat: infrastructure retirement and lock restructure (#1098) - #1112

Merged
jeonghun-jj-lee merged 1 commit into
jj/1091-absorb-opencode-forkfrom
jj/1098-infra-retirement
Sep 14, 2026
Merged

jeonghun-jj-lee merged 1 commit into
jj/1091-absorb-opencode-forkfrom
jj/1098-infra-retirement

Conversation

@jeonghun-jj-lee

Copy link
Copy Markdown
Contributor

Summary

Retires dead fork infrastructure and restructures opencode.lock.json. Phase 4b+5 of the fork-absorption campaign (#1091). Net deletion of 2,455 lines.

Deleted (dead code)

  • opencode_dev.mjs — replaced by build:binary (S2)
  • overlay-sync.mjs, overlay-promotion.mjs, extract_overlay.mjs — sync-from-fork has no source
  • overlay_sync.test.ts, overlay_promotion.test.ts — tests for deleted scripts

Restructured

  • opencode.lock.json — dropped repo, tag, ref, source, platforms; added base_version, base_commit, overlay_hash
  • drift_gate.mjs — stripped to committed-overlay-only integrity check (no fork comparison)
  • fetch_opencode.mjs — removed local-clone, fork-release, platforms validator, AMICODE_OPENCODE_SRC

Migrated (~21 files)

All lock consumers updated to new schema. AMICODE_OPENCODE_SRC and DEFAULT_OPENCODE_PATH fully removed. 18 i18n locale files updated.

Verification

  • 1621 amico-run tests pass
  • 219+ extension tests pass (3 pre-existing failures: skills_integrity, service_runner, clipboard)
  • Zero remaining lock.repo, lock.tag, lock.ref, lock.platforms, AMICODE_OPENCODE_SRC, or DEFAULT_OPENCODE_PATH in src/

Closes #1098. Part of #1091.

Retire dead fork infrastructure and restructure opencode.lock.json:
- Delete overlay-sync, overlay-promotion, extract_overlay, opencode_dev
- Strip drift_gate to committed-overlay-only check
- Restructure lock: drop repo/tag/ref/platforms, add base_commit/overlay_hash
- Migrate all lock consumers to new schema
- Remove AMICODE_OPENCODE_SRC and DEFAULT_OPENCODE_PATH from functional code
- Update i18n placeholders and manifest hashes
- Update test fixtures for the new world

Part of fork-absorption campaign #1091.
@jeonghun-jj-lee jeonghun-jj-lee added the afk Implementable without human interaction label Sep 14, 2026
@coderabbitai

coderabbitai Bot commented Sep 14, 2026 •

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 874d06cb-a397-4423-8c9c-668ee4100954

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@jeonghun-jj-lee
jeonghun-jj-lee marked this pull request as ready for review September 14, 2026 00:23
@jeonghun-jj-lee
jeonghun-jj-lee merged commit 2e6b95b into jj/1091-absorb-opencode-fork Sep 14, 2026
10 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

afk Implementable without human interaction

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant