Skip to content

chore: refresh bug-assess workflow with gh-aw v0.88.7 - #4497

Open
KSchlobohm wants to merge 3 commits into
github:mainfrom
KSchlobohm:chore/upstream-bug-assess-v0.88.7
Open

chore: refresh bug-assess workflow with gh-aw v0.88.7#4497
KSchlobohm wants to merge 3 commits into
github:mainfrom
KSchlobohm:chore/upstream-bug-assess-v0.88.7

Conversation

@KSchlobohm

Copy link
Copy Markdown
Contributor

Summary

Recompile bug-assess with gh-aw v0.88.7, replacing its v0.79.8-generated workflow. The human-maintained bug-assess.md is unchanged: this is a compiler/runtime refresh, not a change to the assessment instructions or intended trigger behavior.

Changes

  • Regenerate .github/workflows/bug-assess.lock.yml with v0.88.7, including the compiler-managed runtime, actions, and container references.
  • Remove the old github/gh-aw-actions/setup@v0.79.8 entry from .github/aw/actions-lock.json; the compiler also reordered an existing entry.
  • Add .github/workflows/*.lock.yml linguist-generated=true to .gitattributes so GitHub recognizes workflow locks as generated files.

No other workflow locks or product source files were changed. The large YAML diff is generated output, not hand-edited workflow logic.

Validation

Local validation in the fork checkout:

gh aw fix bug-assess
  No workflow fixes needed

gh aw compile bug-assess --no-emit --no-check-update
  1 succeeded, 0 warnings

gh aw compile bug-assess --validate
  1 succeeded, 0 warnings

git diff --check
  Passed

The generated change was merged to KSchlobohm/spec-kit's default branch at c39d296da4618485fc679de1b5949be5e442ed4c before live testing. Two separate issues: labeled events then ran the updated workflow on that commit.

Case Test issue Actions run Observed result
Positive: Bob dispatch reports an installed CLI as missing KSchlobohm/spec-kit#2 34400642508 Success; one assessment comment returned Valid / medium; severity-medium applied
Negative: specify --help is claimed to require project initialization KSchlobohm/spec-kit#3 34400654400 Success; one assessment comment returned Invalid / low; invalid applied

All six jobs succeeded in both runs: pre-activation, activation, agent, detection, safe outputs, and conclusion.

These are end-to-end smoke tests of the label trigger, agent execution, assessment comment, and allowed label outputs, not a comprehensive assessment-accuracy evaluation. The fixtures explicitly identify their test purpose. The positive case is based on #4491 and static inspection of the dispatch path; an installed Bob CLI was not exercised locally. No new automated test files were added.

Scope and rollout

Only bug-assess is refreshed; this is not a repository-wide gh aw upgrade. The fork's Issues setting and test labels were configured for testing and are not code changes in this PR. Live issue-triggered testing upstream would use the updated lock after merge to its default branch.

Agent disclosure

Authored and submitted on behalf of @KSchlobohm by GitHub Copilot (model: GPT-6 Astra, autonomous). The commit includes Assisted-by and Co-authored-by trailers. Compilation and the two live fork runs were executed through the agent; no claim of human line-by-line review is made.

Regenerate bug-assess and update compiler-managed metadata.

Assisted-by: GitHub Copilot (model: GPT-6 Astra, autonomous)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI balanced review requested due to automatic review settings September 9, 2026 20:32
@KSchlobohm
KSchlobohm requested a review from mnriem as a code owner September 9, 2026 20:32

Copilot AI left a comment

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.

Copilot review overview

🟡 Changes recommended

The setup action must be restored to the actions lock and regenerated with an immutable SHA pin.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Review tier: Balanced
Findings: 1 High severity · 1 Low severity

New issues introduced by this change (2)
Severity Finding
High severity .github/​aw/​actions-lock.json — This update removes the old setup lock entry without adding one for v0.88.7. The regenerated…
Low severity .gitattributes — The existing rule on line 3 already matches these lock files and sets linguist-generated=true
What changed in this PR

Refreshes bug-assess to gh-aw v0.88.7 and its updated runtime components.

Changes:

  • Regenerates the workflow lock and runtime.
  • Updates the actions lock.
  • Marks workflow locks as generated, though the rule already exists.
File Description
.github/​workflows/​bug-assess.lock.yml Updates generated workflow runtime to v0.88.7.
.github/​aw/​actions-lock.json Removes the previous setup-action pin.
.gitattributes Duplicates the existing generated-file rule.

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .github/aw/actions-lock.json
Comment thread .gitattributes Outdated
Regenerate with gh-aw v0.88.7 and working GitHub authentication so setup references resolve to the release commit.

Assisted-by: GitHub Copilot (model: GPT-6 Astra, autonomous)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings September 9, 2026 20:53
@KSchlobohm

Copy link
Copy Markdown
Contributor Author

Regenerated bug-assess with gh-aw v0.88.7 using working GitHub authentication in commit 0c04cd4. The compiler added the setup action lock entry and pinned all six setup calls to 5e508589e03a7757a7e05b26e834292f5445bfb6; the generated YAML was not hand-edited. This commit addresses the setup pin only; the separate .gitattributes finding remains open.

Posted on behalf of @KSchlobohm by GitHub Copilot (model: GPT-6 Astra, autonomous).

Restore .gitattributes to its pre-PR contents while retaining the existing generated-workflow attributes.

Assisted-by: GitHub Copilot (model: GPT-6 Astra, autonomous)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot AI left a comment

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.

Copilot review overview

🔵 Needs a closer look

The generated runtime and security architecture changed substantially despite successful validation and smoke tests.

Review tier: Balanced
Findings: None

Issues resolved since last review (2)
Severity Finding
Low severity .gitattributes — The existing rule on line 3 already matches these lock files and sets linguist-generated=trueView resolved comment
High severity .github/​aw/​actions-lock.json — This update removes the old setup lock entry without adding one for v0.88.7. The regenerated… View resolved comment

Copilot AI review requested due to automatic review settings September 9, 2026 20:56

Copilot AI left a comment

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.

Copilot review overview

🟢 Approval recommended

The generated refresh is internally consistent and both positive and negative end-to-end smoke runs succeeded.

Review tier: Balanced
Findings: None

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.

2 participants