EnterpriseAI/ClosedCode
1
0
Fork 0
Code Issues Pull Requests Actions 32 Packages Projects Releases Wiki Activity

feat(cli): log updater check decisions

Browse Source 
Add Effect.logInfo at every previously-silent path in the updater
check so each run is observable in opencode.log: skip reasons
(local install, disabled, policy), the resolved version with source
(cached vs registry), and the final action (up-to-date, dismissed,
declined).
This commit is contained in:
Dax Raad
2026-06-27 13:29:46 -04:00
parent 7edd6c3a1d
commit b458dd8c63
1 changed files with 24 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/cli/src/services/updater.ts
+19 -7
View File
@@ -165,34 +165,46 @@ export const layer = Layer.effect(
) )
const check = Effect.fn("cli.updater.check")(function* (options: { interactive: boolean }) { const check = Effect.fn("cli.updater.check")(function* (options: { interactive: boolean }) {
if (InstallationLocal || Flag.OPENCODE_DISABLE_AUTOUPDATE) return if (InstallationLocal || Flag.OPENCODE_DISABLE_AUTOUPDATE)
return yield* Effect.logInfo("update check skipped", {
reason: InstallationLocal ? "local-install" : "disabled",
version: InstallationVersion,
channel: InstallationChannel,
})
const policy = yield* readPolicy() const policy = yield* readPolicy()
if (policy === false) return if (policy === false) return yield* Effect.logInfo("update check skipped", { reason: "policy-disabled" })
return yield* Effect.scoped( return yield* Effect.scoped(
Effect.gen(function* () { Effect.gen(function* () {
yield* Flock.effect(`opencode-cli-updater-${channel}`, { dir: path.join(global.state, "locks") }) yield* Flock.effect(`opencode-cli-updater-${channel}`, { dir: path.join(global.state, "locks") })
const previous = yield* readState() const previous = yield* readState()
const now = Date.now() const now = Date.now()
const version = const cached = previous && now - previous.checked < checkInterval
previous && now - previous.checked < checkInterval const version = cached
? previous.latest ? previous!.latest
: yield* latest().pipe( : yield* latest().pipe(
Effect.tap((value) => Effect.tap((value) =>
writeState({ checked: now, latest: value, dismissed: previous?.dismissed }), writeState({ checked: now, latest: value, dismissed: previous?.dismissed }),
), ),
) )
yield* Effect.logInfo("update check", {
current: InstallationVersion,
latest: version,
source: cached ? "cached" : "registry",
})
const next = action( const next = action(
InstallationVersion, InstallationVersion,
version, version,
Flag.OPENCODE_ALWAYS_NOTIFY_UPDATE ? "notify" : policy, Flag.OPENCODE_ALWAYS_NOTIFY_UPDATE ? "notify" : policy,
options.interactive && process.stdin.isTTY && process.stdout.isTTY, options.interactive && process.stdin.isTTY && process.stdout.isTTY,
) )
if (next === "none" || (next === "confirm" && previous?.dismissed === version)) return if (next === "none") return yield* Effect.logInfo("update check done", { action: "up-to-date" })
if (next === "confirm" && previous?.dismissed === version)
return yield* Effect.logInfo("update check done", { action: "dismissed", version })
const install = next === "upgrade" || (yield* confirm(version)) const install = next === "upgrade" || (yield* confirm(version))
if (!install) { if (!install) {
yield* writeState({ checked: previous?.checked ?? now, latest: version, dismissed: version }) yield* writeState({ checked: previous?.checked ?? now, latest: version, dismissed: version })
return return yield* Effect.logInfo("update check done", { action: "declined", version })
} }
const detected = yield* method() const detected = yield* method()
if (!detected) return yield* Effect.logWarning("automatic update skipped: installation method not found") if (!detected) return yield* Effect.logWarning("automatic update skipped: installation method not found")