Skip to content

ci(goal): prove pinned OpenCode 2.1.4 native lifecycle and restart (#283) - #285

Merged
ByBrawe merged 2 commits into
mainfrom
test/v2.1.4-real-native-contract-20261027
Oct 27, 2026
Merged

ByBrawe merged 2 commits into
mainfrom
test/v2.1.4-real-native-contract-20261027

Conversation

@ByBrawe

@ByBrawe ByBrawe commented Oct 27, 2026

Copy link
Copy Markdown
Owner

Scope

The current Goal matrix labels a pinned OpenCode 2.0.16 job as "current stable" and tests a moving latest host separately. OpenCode 2.1.4 changes provider-replay and compaction data contracts; a reproducible pinned 2.1.4 real native lane is needed before broad Goal #276 and release upgrades.

Changes

  • Accurately relabel the 2.0.16 lane as pinned legacy V2 compatibility.
  • Add Ubuntu Linux real OpenCode 2.1.4 lane with exact CLI/version guard; clean npm ci + Goal build.
  • Execute existing native /goal host lifecycle, waiting-user and ownership canary, plus same-session re-arm after server restart; prove against actual host, not only mocks.
  • Keep moving latest and old 2.0.16 lanes; do not silently upgrade V1 or claim Windows Desktop GUI was tested.

Merge criteria

All PR CI/real-host checks pass and exact 2.1.4 canary produces successful evidence. If host 2.1.4 fails, fix the actual native contract/test and preserve the failure signal; no bypass. This closes only the stable Ubuntu slice of Goal #283, not the entire dev/Windows/macOS matrix. Cross-plugin issue Loop #185.

Copilot AI balanced review requested due to automatic review settings October 27, 2026 00:05

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-27T00:07:00.422755Z 3c87a19 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 3c87a19ebf

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

node-version: "24"
- name: Install exact repository dependencies and build native plugin
run: |
npm ci

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Use an install command that works without a lockfile

This new job always stops here because commit 8674d44 tracks neither package-lock.json nor npm-shrinkwrap.json. Running the exact command reports EUSAGE with npm's explanation that npm ci can only install when one of those lockfiles exists, so the pinned 2.1.4 lifecycle and restart steps never execute. Commit a lockfile or retain npm install for this lane.

Useful? React with 👍 / 👎.

@ByBrawe
ByBrawe merged commit c70c127 into main Oct 27, 2026
27 checks passed
@github-actions
github-actions Bot deleted the test/v2.1.4-real-native-contract-20261027 branch October 27, 2026 00:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants