[sonichi]

Priority

Level: high
Reason: main CI has been red since the #5815 merge (3f38f18); this unblocks every open PR's python test shards.

Closes

N/A. CI repair. The same one-line change was first found and carried in #5851, which will rebase onto this PR.

Summary

tests/gateway-credential-sources.test.py reads EnvFiles.credential_values, a name no source defines; the field the owner module defines is credential_pairs. Guard.test_a_declared_bridge_reads_no_credential_file_and_the_host_resolves_what_it_carries errors in both subtests (package, src), and tests/hermetic-env-sentinels.test.py, which runs that suite, fails with it. This renames the one attribute access. No source change.

Checklist

• Confirmed no other open PR carries this fix, apart from fix(gateway-credentials): refuse credential-shaped gateway URLs, cap the declaration check, report an unlistable agents dir #5851
• Git author email matches my CLA-signed email
• Single concern
• Confirmed the failure exists on main (the main CI run at 3f38f18 failed; output below)
• Test added: N/A. This edits an existing test so it runs.
• Before/after evidence below
• Live path? N/A. Test-only.
• Stacked PR? N/A
• Hardcoded-path scan: no paths added

Docs: N/A, test-only.

Before / after evidence

Before (origin/main 3f38f18):

ERROR: test_a_declared_bridge_reads_no_credential_file_and_the_host_resolves_what_it_carries (__main__.Guard...) [package]
AttributeError: 'EnvFiles' object has no attribute 'credential_values'. Did you mean: 'credential_pairs'?
ERROR: ... [src]
FAILED (errors=2)

After (this head):

python3 tests/gateway-credential-sources.test.py                     Ran 153 tests ... OK
python3 tests/hermetic-env-sentinels.test.py Sentinels.test_the_credential_owner_suite_reads_no_outer_selector   OK

Test plan

I ran both commands above under env -i HOME=$HOME PATH=$PATH nice -n 19. Edge cases, migrations and risks: N/A.

🤖 Generated with Claude Code

[github-actions]

Coverage Gate

✅ Diff coverage PASSES the 95% bar. Whole-tree (informational): 94%.

Diff Coverage

Diff: origin/main...HEAD, staged and unstaged changes

No lines with coverage information in this diff.

[qingyun-wu]

Reviewed at 8914f9c: one-line test fix. EnvFiles on main defines credential_pairs (src/gateway_credential_sources.py:280, same in the vendored packages/ag2-sparrow copy, and register_pairs reads it at :488); the assertion named a field that never existed after #5815 landed. Approving; this unblocks main.

[sonichi]

@cla-assistant check

[sonichi]

Covered by #5851 (merged as cf87f32), which carried the same one-line credential_pairs test fix. Closing because this PR's CLA status never posted.