Skip to content

feat(add package): opencode#1857

Open
mvanhorn wants to merge 1 commit intowimpysworld:mainfrom
mvanhorn:osc/issue-1856-opencode
Open

feat(add package): opencode#1857
mvanhorn wants to merge 1 commit intowimpysworld:mainfrom
mvanhorn:osc/issue-1856-opencode

Conversation

@mvanhorn
Copy link
Copy Markdown
Contributor

Resolves #1856.

opencode is an open source coding agent that publishes authoritative .deb packages for amd64 and arm64 from GitHub Releases at anomalyco/opencode (the new home of sst/opencode; the old URL 301-redirects to the new one).

Package definition derived from @zackattackz's draft in #1856.

Validation

Verified the URL extraction logic against the live JSON for v1.14.25 (pretty-printed via jq, as get_github_releases does):

HOST_ARCH=amd64 -> URL=...releases/download/v1.14.25/opencode-desktop-linux-amd64.deb -> VERSION_PUBLISHED=1.14.25
HOST_ARCH=arm64 -> URL=...releases/download/v1.14.25/opencode-desktop-linux-arm64.deb -> VERSION_PUBLISHED=1.14.25

The grep pattern terminates with \.deb" so it does not match the .deb.sig signature assets, the opencode-electron-linux-*.deb Electron variants, or the .rpm Fedora assets in the same release.

Resolves wimpysworld#1856.

opencode is an open source coding agent that publishes signed .deb
packages for amd64 and arm64 to GitHub Releases at anomalyco/opencode.

Package definition based on @zackattackz's draft in wimpysworld#1856.
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

1 issue found across 1 file

Confidence score: 3/5

  • There is a concrete integration risk: the new package in 01-main/packages/opencode is not registered in 01-main/manifest, so manifest-driven discovery flows may not find it at runtime.
  • Given the high severity/confidence signal (7/10, 9/10), this looks user-impacting rather than a minor housekeeping gap, which is why the score is moderate instead of merge-safe.
  • This is likely straightforward to fix before merge by adding the required manifest registration, which should reduce regression risk quickly.
  • Pay close attention to 01-main/packages/opencode, 01-main/manifest - missing registration can make the package undiscoverable in manifest-based paths.
Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="01-main/packages/opencode">

<violation number="1" location="01-main/packages/opencode:1">
P1: New package definition is missing required registration in `01-main/manifest`, which can make it undiscoverable to manifest-driven flows.</violation>
</file>

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.

Comment thread 01-main/packages/opencode
@@ -0,0 +1,10 @@
DEFVER=1
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P1: New package definition is missing required registration in 01-main/manifest, which can make it undiscoverable to manifest-driven flows.

Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At 01-main/packages/opencode, line 1:

<comment>New package definition is missing required registration in `01-main/manifest`, which can make it undiscoverable to manifest-driven flows.</comment>

<file context>
@@ -0,0 +1,10 @@
+DEFVER=1
+ARCHS_SUPPORTED="amd64 arm64"
+get_github_releases "anomalyco/opencode" "latest"
</file context>

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.

App Request: OpenCode

1 participant