[opencode-agent]

Summary

I've successfully fixed the hardcoded registry issue in packages/opencode/src/installation/index.ts. Here's what was changed:

Changes made:

1. Added getRegistryUrl() function (lines 165-169): This function retrieves the user's configured npm registry using npm config get registry. If no registry is configured, it defaults to https://registry.npmjs.org.

2. Updated latest() function (lines 171-194):

   • Now calls getRegistryUrl() to get the user's configured registry
   • Normalizes the registry URL by removing trailing slashes (to prevent double slashes in the final URL)
   • Attempts to fetch the latest version from the configured registry
   • Fallback mechanism: If the configured registry fails (e.g., a network error or 404), and it's not already the default registry, it falls back to registry.npmjs.org
   • If both the configured registry and the fallback fail, it throws an error

Key features:

• ✅ Respects user's npm configuration
• ✅ Falls back to registry.npmjs.org if the configured registry fails
• ✅ Handles trailing slashes in registry URLs properly
• ✅ Maintains backward compatibility with the default registry

Closes #4830

opencode session  |  github run

[rekram1-node]

/oc drop this: .catch(() => {
// Fallback to default registry if configured registry fails
if (registry !== "https://registry.npmjs.org") {
return fetch(https://registry.npmjs.org/opencode-ai/${channel})
.then((res) => {
if (!res.ok) throw new Error(res.statusText)
return res.json()
})
.then((data: any) => data.version)
}
throw new Error("Failed to fetch latest version")
})

  there isn't much of a reason for it

[opencode-agent]

Done! I've simplified the latest() function by removing the .catch() fallback block and converting it to use await instead of promise chains. The function now respects the user's npm configuration without the unnecessary fallback logic.

opencode session  |  github run