Skip to content

fix(verifier): accept bounded API key placeholders - #142

Merged
tiammomo merged 1 commit into
mainfrom
tiammomo/fix/api-key-template-placeholders
Sep 8, 2026
Merged

fix(verifier): accept bounded API key placeholders#142
tiammomo merged 1 commit into
mainfrom
tiammomo/fix/api-key-template-placeholders

Conversation

@tiammomo

@tiammomo tiammomo commented Sep 8, 2026

Copy link
Copy Markdown
Owner

Closes #141

Accept strict your-provider-api-key placeholders in tracked environment templates so public example files can enter isolated verification.


Extend the bounded full-match placeholder policy alongside the existing replace-with format. Preserve all filename, encoding, size, regular-file, directory and sensitive-value checks. Regression coverage includes public provider placeholders, multi-part provider names, the exact 128-character boundary, and malformed or sensitive-looking values. The new regression failed before the fix; isolated pre-commit validation passed all 603 tests, lint, formatting, CLI smoke and build.

Verified with uv run --python 3.12 python -W error::ResourceWarning -m unittest discover -s tests -v, uvx ruff check ., uvx ruff format --check ., uv run reposteward --help, uv build --offline.

Areas for careful review:

  • No known behavior changes outside the reported bug.

Implementation assistance: an external coding workspace was used to prepare the change and its
tests. tiammomo reviewed the final diff, understands it, and takes responsibility
for this contribution.

Closes #141

Signed-off-by: tiammomo <26957354+tiammomo@users.noreply.github.com>
@tiammomo
tiammomo marked this pull request as ready for review September 8, 2026 03:32
@tiammomo
tiammomo merged commit a198b11 into main Sep 8, 2026
1 check passed
@tiammomo
tiammomo deleted the tiammomo/fix/api-key-template-placeholders branch September 8, 2026 03:36
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.

fix(verifier): accept bounded your-provider-api-key template placeholders

1 participant