Skip to content

Fix bounded query smoke workflow run - #6755

Merged
lpcox merged 4 commits into
mainfrom
fix-bounded-query-smoke-run
Jul 30, 2026
Merged

Fix bounded query smoke workflow run#6755
lpcox merged 4 commits into
mainfrom
fix-bounded-query-smoke-run

Conversation

@lpcox

@lpcox lpcox commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • persist bounded-query broker audit records synchronously so failure diagnostics survive immediate teardown
  • bootstrap the latest published AWF release, then replace it with the PR checkout's unreleased AWF build before agent execution
  • build unreleased AWF container images locally and avoid incompatible generated pull flags
  • add regression coverage for immediate audit durability

Validation

The branch-dispatched smoke-bounded-queries workflow completed successfully. The deterministic confidentiality-budget matrix, agent bounded query, protected audit assertion, safe outputs, and conclusion jobs all passed.

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

Copilot-Session: 6923ed6e-0ce2-4fa9-aee9-ccff99b90b90
Copilot AI review requested due to automatic review settings July 30, 2026 22:34

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.

Pull request overview

Makes bounded-query audit diagnostics immediately available during rapid broker teardown.

Changes:

  • Replaces buffered audit writes with synchronous file writes.
  • Adds regression coverage for immediate audit visibility.
Show a summary per file
File Description
containers/bounded-query/broker/audit.js Writes audit records synchronously with stderr fallback.
src/bounded-query/broker.test.ts Verifies records are readable immediately after writing.

Review details

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Balanced

@github-actions

Copy link
Copy Markdown
Contributor

✅ Coverage Check Passed

Overall Coverage

Metric Base PR Delta
Lines 99.13% 99.16% 📈 +0.03%
Statements 99.04% 99.07% 📈 +0.03%
Functions 99.28% 99.28% ➡️ +0.00%
Branches 95.87% 95.87% ➡️ +0.00%
📁 Per-file Coverage Changes (1 files)
File Lines (Before → After) Statements (Before → After)
src/log-directory-setup.ts 96.2% → 100.0% (+3.78%) 96.3% → 100.0% (+3.71%)

Coverage comparison generated by scripts/ci/compare-coverage.ts

Bootstrap the latest published AWF release.

Replace it with the workflow checkout build before the agent starts.

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

Copilot-Session: 6923ed6e-0ce2-4fa9-aee9-ccff99b90b90
@github-actions

Copy link
Copy Markdown
Contributor

✅ Copilot review passed with no inline comments.

@lpcox Add the ready-for-aw label to this PR to trigger agentic CI smoke tests.

lpcox added 2 commits July 30, 2026 16:05
Avoid pulling unreleased AWF images.

Build the agent sandbox images from the workflow checkout instead.

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

Copilot-Session: 6923ed6e-0ce2-4fa9-aee9-ccff99b90b90
Remove the generated skip-pull flag that conflicts with build-local.

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

Copilot-Session: 6923ed6e-0ce2-4fa9-aee9-ccff99b90b90
@lpcox
lpcox merged commit ef8b73c into main Jul 30, 2026
30 of 33 checks passed
@lpcox
lpcox deleted the fix-bounded-query-smoke-run branch July 30, 2026 23:33
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