Skip to content

fix: the audit's token scope was one folded string, so both tokens were empty - #22

Merged
MarketDataApp merged 1 commit into
mainfrom
fix/audit-token-repository-list
Sep 22, 2026
Merged

MarketDataApp merged 1 commit into
mainfrom
fix/audit-token-repository-list

fix: the audit's token scope was one folded string, so both tokens we…

4978fec
Select commit
Loading
Failed to load commit list.
marketdata-code-review / Code review succeeded Sep 22, 2026 in 0s

✅ Ready

This change switches both GitHub App repository lists from folded to literal YAML scalars so the token action receives separate repository names. The regression test now parses the values using the action's separators and rejects folded, space-joined names.

Annotations

Check notice on line 581 in tests/test_callers.py

See this annotation in the file changed.

@marketdata-code-review marketdata-code-review / Code review

Move incident history out of the test

The new comment records the specific 404 incident and empty-token outcome rather than only the parsing constraint. Keep the separator rationale in the test and leave the incident narrative in the pull request or commit message.