Skip to content

Add Playwright authentication resilience - #470

Merged
edithatogo merged 3 commits into
masterfrom
codex/t15-browser-fallback
Aug 8, 2026
Merged

Add Playwright authentication resilience#470
edithatogo merged 3 commits into
masterfrom
codex/t15-browser-fallback

Conversation

@edithatogo

Copy link
Copy Markdown
Owner

Summary

Validation

  • Biome CI passed
  • TypeScript typecheck and build passed
  • 115 test files / 902 tests passed
  • production dependency audit found 0 vulnerabilities
  • git diff --check passed

Security boundaries

No real session state was captured or committed. Browser launch is never implicit, write requests never use this fallback, and challenge completion remains an explicit local human action.

Closes #469

Copilot AI lite review requested due to automatic review settings August 8, 2026 13:13
@cursor

cursor Bot commented Aug 8, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@codecov

codecov Bot commented Aug 8, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@edithatogo
edithatogo merged commit d51a860 into master Aug 8, 2026
20 of 21 checks passed
@edithatogo
edithatogo deleted the codex/t15-browser-fallback branch August 8, 2026 13:56
edithatogo added a commit that referenced this pull request Aug 9, 2026
## Summary
- move completed T13-T16 tracks into the canonical Conductor archive
- mark every closeout task complete and metadata archived
- add hosted verification receipts for PRs #464, #467, #470, and #475
- preserve P14/P15 external credential activation gates

## Validation
- all archive index and registry links resolve
- all four archived plans contain zero pending tasks
- all four metadata files report `archived`
- `git diff --check`
- source/runtime code and dependencies are unchanged

## Hosted evidence
All comments, inline comments, reviews, and checks on the four
implementation PRs were re-inspected. The Node 26 warning on #464 was
considered and rejected against successful hosted Node 26 execution.

---------

Co-authored-by: Dylan Mordaunt (ISLHD) <Dylan.Mordaunt@health.nsw.gov.au>
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.

T15-01 Add Playwright authentication fallback and storage-state refresh

3 participants