[pr-review] Daily PR Code Quality Review — 31297788393

[github-actions]

Overview

Daily code-quality scan of the 5 most recently opened PRs in github/gh-aw (ordered by created_at desc). Focus: missing Go error handling, undocumented exports, assertion-free tests, and oversized functions (>80 lines).

PR	Author	Signal	Top issues
#51533 [WIP] Update static analysis report for 2026-08-09	Copilot	🟢	No reviewable diff yet (draft WIP, 0 files changed)
#51532 [WIP] Fix validation logic for require-invalid-date-check-before-compare rule	Copilot	🟢	No reviewable diff yet (draft WIP, 0 files changed)
#51517 Refactor add interactive workflow run prompt	Copilot	🟢	No issues
#51514 fix(seenmapbool): treat non-true values in declaration-site composite literals as non-set maps	Copilot	🟢	No issues
#51508 [log] Add debug logging to under-instrumented wasm/console helpers	github-actions[bot]	🟢	1 style nit (see below)

Per-PR notes

#51533 / #51532 — WIP drafts

• Both are draft Copilot agent PRs created moments ago.
• get_diff / get_files returned empty; no code to assess for error handling, docs, tests, or function size.
• Re-review once commits land.

#51517 — interactive workflow run refactor

• Splits checkStatusAndOfferRun into focused helpers (waitForWorkflowStatus, runAddedWorkflowOnce, etc.); bodies stay well under 80 lines.
• Error paths keep if err != nil handling; run-prompt cancel is intentionally non-fatal (return nil).
• New helpers are unexported; no missing exported doc comments.
• New table test uses assert.Equal — not assertion-free.
• Verdict: clean structural refactor.

#51514 — seenmapbool literal fix

• Adds markIfNonSetLiteral with a clear doc comment; logic is small and nil-safe on ObjectOf.
• Analyzer testdata adds two “good” cases (false in composite literals) without needing // want markers.
• No new exported APIs; no oversized functions.
• Verdict: focused lint fix with appropriate fixtures.

#51508 — debug logging

• Logging-only changes across colorwriter, console wasm, parser wasm, and awf_helpers; existing error checks preserved.
• Exported APIs already documented; no new exports.
• Nit: several call sites use Printf/Print rather than structured Debug("msg", "key", value) style preferred in AGENTS.md / pkg/logger guidance (≤1 issue).
• No new tests (acceptable for pure instrumentation).

Summary

All five recent open PRs rate 🟢. Two are empty WIP shells; the three with diffs look solid (refactor + linter fix + logging). Only actionable follow-up is the optional logger-style nit on #51508.

References:

• Run §31297788393
• #51517
• #51514

Warning

Firewall blocked 2 domains

The following domains were blocked by the firewall during workflow execution:

• registry.npmjs.org
• repo42.cursor.sh

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "registry.npmjs.org"
    - "repo42.cursor.sh"

See Network Configuration for more information.

Generated by 🖱️ Daily PR Code Quality Review — Cursor · ◷

• expires on Aug 9, 2026, 10:00 PM UTC-08:00

[github-actions]

This issue is being closed as outdated. A newer issue has been created: #51704

View newer issue

---

This action was performed automatically by the Daily PR Code Quality Review — Cursor workflow.