Skip to content

fix(SAFE-T1004): replace placeholder guidance and add rule tests#185

Open
bishnubista wants to merge 1 commit intosafe-agentic-framework:mainfrom
bishnubista:codex/fix-safe-t1004-quality
Open

fix(SAFE-T1004): replace placeholder guidance and add rule tests#185
bishnubista wants to merge 1 commit intosafe-agentic-framework:mainfrom
bishnubista:codex/fix-safe-t1004-quality

Conversation

@bishnubista
Copy link
Collaborator

@bishnubista bishnubista commented Feb 24, 2026

Summary

Harden SAFE-T1004 technique documentation, detection rule, and test coverage:

  • Rewrite README to remove placeholder mitigations and incorrect related-technique mappings
  • Update detection rule to use actionable ODR matching and valid SAFE-MCP reference URLs
  • Relax unsupported ARP event-id assumption in detection logic
  • Expand test logs with positive/negative cases
  • Add runnable test_detection_rule.py validation script

Type of Contribution

  • New Technique
  • New Mitigation
  • Update to existing content
  • Documentation improvement

Checklist

Validation

  • python3 techniques/SAFE-T1004/test_detection_rule.py
  • Parsed techniques/SAFE-T1004/detection-rule.yml with yaml.safe_load
  • Checked relative links in techniques/SAFE-T1004/README.md

Related Issues

N/A

@bishnubista bishnubista force-pushed the codex/fix-safe-t1004-quality branch from 91b0bc8 to f454833 Compare February 24, 2026 06:27
- Rewrite SAFE-T1004 README to remove placeholder mitigations and
  incorrect related-technique mappings
- Update detection rule to use actionable ODR matching and valid
  SAFE-MCP reference URLs
- Relax unsupported ARP event-id assumption in detection logic
- Expand test logs with positive/negative cases
- Add runnable test_detection_rule.py validation script

Signed-off-by: bishnubista <bista.developer@gmail.com>
@bishnubista bishnubista force-pushed the codex/fix-safe-t1004-quality branch from f454833 to ac27c5f Compare February 24, 2026 06:33
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.

1 participant