Skip to content

allow github.event.inputs.*#54

Merged
dsyme merged 1 commit intomainfrom
p3
Aug 14, 2025
Merged

allow github.event.inputs.*#54
dsyme merged 1 commit intomainfrom
p3

Conversation

@dsyme
Copy link
Contributor

@dsyme dsyme commented Aug 14, 2025

@dsyme dsyme merged commit 9a62edb into main Aug 14, 2025
6 checks passed
@dsyme dsyme deleted the p3 branch August 14, 2025 13:03
pelikhan added a commit that referenced this pull request Sep 4, 2025
…tion, workflow name defaults, GitHub Code Scanning integration, custom rule ID support, and test workflows (#54) (#310)

* Initial plan

* Implement create-security-report safe output feature



* Final implementation with schema fix, formatting, and validation



* Implement PR feedback: configurable driver, workflow filename rule IDs, and optional column support



* Default security report driver to agentic workflow name from frontmatter



* Add support for optional ruleIdSuffix in security reports

Allow LLMs to provide custom rule ID suffixes in security reports via the ruleIdSuffix field. When not provided, defaults to the existing number scheme.

- Add ruleIdSuffix validation (alphanumeric, hyphens, underscores only)
- Update rule ID generation to use custom suffix when available
- Add comprehensive tests for custom and default rule ID scenarios
- Update documentation to describe new functionality
- Maintain backward compatibility with existing workflows



* Add test agentic workflows for create-security-report safe output

Added Claude and Codex test workflows to validate the new create-security-report safe output functionality



* Update test workflows to use workflow_dispatch trigger instead of issues



---------

Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
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.

2 participants