Skip to content

chore: add token limit env inputs - #33

Merged
jescalada merged 2 commits into
mainfrom
ai-automated-issue-and-pr-assessment
Aug 10, 2026
Merged

chore: add token limit env inputs#33
jescalada merged 2 commits into
mainfrom
ai-automated-issue-and-pr-assessment

Conversation

@jescalada

Copy link
Copy Markdown
Owner

No description provided.

@github-actions

github-actions Bot commented Aug 10, 2026

Copy link
Copy Markdown

Automated Security Review

Summary

No security issues found.

The change only passes two non-sensitive configuration variables (MAX_OUTPUT_TOKENS, TOKEN_BUDGET) from repository vars into the workflow environment and appends their values to a debug section of the system prompt. No secrets are exposed, no user-controlled input reaches a shell, SQL, or deserialization sink, and the values are read via os.environ.get with safe defaults and interpolated into a string only.

Disclaimer: This review is AI-generated. Please validate the findings before fixing.

Reviewed by claude-opus-5. Re-run by commenting /security-review on this PR.

@jescalada

Copy link
Copy Markdown
Owner Author

/security-review

@jescalada
jescalada merged commit 60ac294 into main Aug 10, 2026
14 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant