[codex] add ai-assisted detection demo and portfolio docs#18
Merged
Conversation
6614959 to
96a23c0
Compare
Includes the rebased headless plotting backend fix needed for CI after updating onto main.
96a23c0 to
5e7154b
Compare
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR turns the repository into a clearer two-demo
telemetry-labportfolio artifact and adds a constrainedai-assisted-detection-demoalongside the existingtelemetry-window-demo.The new demo is intentionally framed as a small, reviewable security engineering prototype. Detection, grouping, and ATT&CK attachment stay deterministic. The LLM stage is limited to JSON-only case drafting, requires human verification, and is fail-closed on invalid or unsafe output.
Problem
Before this change, the repository only presented the time-series telemetry workflow, which made the broader telemetry-to-detection story hard to see from the landing page. There was also no second demo showing how deterministic telemetry processing could feed an auditable AI-assisted SOC-style workflow.
For a public portfolio repository, that made the project read as a single prototype instead of a coherent lab with multiple small, credible demos.
Root cause
The repo structure, README narrative, CLI surface, and test coverage had not yet been extended to represent the second demo as a first-class artifact. The AI-assisted detection pipeline, its guardrails, and its audit semantics needed to be encoded in code, artifacts, tests, and docs rather than implied.
Fix
This PR adds
demos/ai-assisted-detection-demowith:It also adds the corresponding Python implementation under
src/telemetry_window_demo/ai_assisted_detection_demo/and exposes the demo throughpython -m telemetry_window_demo.cli run-ai-demo.The pipeline is hardened to behave conservatively:
case_idare rejectedThe docs are updated so the repository reads clearly as a two-demo project. The top-level README now includes a concise comparison matrix, the AI-assisted demo README includes accepted / rejected / degraded reviewer walkthroughs, and the design doc states the lifecycle contract and audit schema version explicitly.
Validation
I ran the full test suite after the final documentation and integration pass:
python -m pytest->31 passedI also verified that the documentation and generated artifacts use the same audit schema version string:
ai-assisted-detection-audit/v1Notes
This remains a portfolio-grade prototype, not a production SOC system. The LLM does not make final incident decisions, does not modify rules, and does not execute response actions.