[pr-review] Daily PR Code Quality Review — 38210088131 #67911

#67894

• Mostly regenerated .lock.yml files plus an updated pkg/workflow/shared_go_make_test.go.
• Large diff (about 72 KB) was not fully read, so no Go-specific findings were confirmed.
• No issues found.

#67890

• Changes are small: astutil.UnwrapParenExpr is applied in 5 linters and the shared MatchDiscardedErrorCall.
• Adds table-driven TestMatchDiscardedErrorCallParenthesized; it asserts on ok, call, package path, and function name.
• The new function is not exported and no new exported API appears in the diff. UnwrapParenExpr already exists.
• Minor: the testdata duplicates the (f.Close()) and (json.Marshal(f)) lines. This is harmless but redundant.
• No issues beyond that.

#67889

• Mostly workflow markdown and .lock.yml changes. No Go code in the files reviewed.
• No issues found.

#67886

• Default version pin updates only. No Go code reviewed.
• No issues found.

#67884

• Touches JS (.cjs) files, including daily_aic_scan.cjs, daily_aic_api_budget.cjs, and restore_aic_scan_cache.cjs. Each changed file has an accompanying or updated test file.
• The Go-specific checks (error handling, doc comments) do not apply.
• Possible concern: very large diff (about 111 KB), which is hard to review. Splitting it would help reviewers.