Skip to content

Add bounded authentication-only diagnosis for Global review - #240

Merged
Pigbibi merged 2 commits into
mainfrom
fix/global-etf-auth-diagnostic
Sep 16, 2026
Merged

Pigbibi merged 2 commits into
mainfrom
fix/global-etf-auth-diagnostic

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

The Global review authentication preflight returned only a generic failure, so the failed request could not be attributed to token acquisition or the audit service. Add bounded, credential-free classifications and an auth-only dispatch that cannot enter research or model execution.

The default remains plan-only; execute and auth-only are mutually exclusive. HTTP diagnostics expose only a fixed stage and numeric status, never URLs, query strings, tokens, response bodies or exception text.

Validation: 27 focused tests passed, 1 skipped, 17 subtests; actionlint, Ruff, Bash syntax and diff checks passed. Existing failed research state remains unchanged.

Pigbibi and others added 2 commits September 17, 2026 02:26
…ponse text

Co-Authored-By: Codex <noreply@openai.com>
@Pigbibi
Pigbibi merged commit a9f5ed1 into main Sep 16, 2026
4 checks passed
@Pigbibi
Pigbibi deleted the fix/global-etf-auth-diagnostic branch September 16, 2026 18: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.

1 participant