Skip to content

[codex] add machine-readable run summaries#15

Merged
stacknil merged 2 commits intomainfrom
codex/issue-10-summary-artifact
Mar 25, 2026
Merged

[codex] add machine-readable run summaries#15
stacknil merged 2 commits intomainfrom
codex/issue-10-summary-artifact

Conversation

@stacknil
Copy link
Copy Markdown
Owner

Summary

  • emit a compact summary.json for each pipeline run alongside the existing CSV and PNG outputs
  • include the summary artifact in both bundled sample output directories
  • add focused e2e checks for summary creation and core fields on the default and richer sample paths

Summary fields

  • input path
  • output directory
  • normalized event count
  • window count
  • feature row count
  • alert count
  • triggered rule names
  • triggered rule counts
  • cooldown seconds
  • generated artifact paths

Validation

  • pytest

Notes

  • both configs/default.yaml and configs/richer_sample.yaml now produce summary.json
  • existing default and richer sample CSV/PNG outputs are unchanged; this PR adds the summary artifact beside them

Fixes #10

@stacknil stacknil added enhancement New feature or request docs Documentation and README behavior labels Mar 25, 2026
@stacknil stacknil added this to the v0.3 milestone Mar 25, 2026
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 8b2e937c16

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@stacknil
Copy link
Copy Markdown
Owner Author

@codex review
Focus on:

  • whether null
    ules is now handled safely in the summary-writing path
  • whether the new test covers the regression without widening scope

@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@stacknil stacknil merged commit c244023 into main Mar 25, 2026
2 checks passed
@stacknil stacknil deleted the codex/issue-10-summary-artifact branch March 25, 2026 05:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation and README behavior enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

v0.3: add a compact machine-readable summary artifact

1 participant