Skip to content

Add approved resurfacing digest routing#55

Merged
WalksWithASwagger merged 1 commit into
mainfrom
codex/issue-51-digest-approved-routing
May 22, 2026
Merged

Add approved resurfacing digest routing#55
WalksWithASwagger merged 1 commit into
mainfrom
codex/issue-51-digest-approved-routing

Conversation

@WalksWithASwagger
Copy link
Copy Markdown
Owner

Summary

  • Keep digest generation report-only by default and require explicit approval for routing.
  • Add approved local follow-up queue routing plus local Notion page/task draft routing.
  • Add CLI flags and tests for approval, dry-run, missing config, and approved write paths.

Verification

  • PYTHON=/Users/kk/Code/spektorAI/venv/bin/python make lint
  • PYTHON=/Users/kk/Code/spektorAI/venv/bin/python make test -> 288 passed
  • PYTHON=/Users/kk/Code/spektorAI/venv/bin/python make digest
  • git diff --check

Closes #51

@github-actions
Copy link
Copy Markdown

Agentic PR Review

Verdict: needs-human

Checks:

  • Linked issue via Closes #...: yes
  • Issue acceptance criteria found: 4
  • PR verification section present: yes
  • PR acceptance self-check present: no
  • PR self-check covers issue acceptance: no
  • Diff within v1 limits: yes
  • Missing acceptance coverage:
    • Digest routing remains report-only until an explicit approve action is taken.
    • Approved destinations support at least local follow-up queue and Notion task/page draft paths.
    • Dry-run and missing-config behavior is visible before any outbound write.
    • Tests cover approved write, dry-run, and missing-config paths.

This PR is not blocked from human review, but the agentic gate is not satisfied.

@github-actions github-actions Bot added the needs-human Agentic automation stopped for human judgment. label May 22, 2026
@WalksWithASwagger WalksWithASwagger merged commit c96a6e5 into main May 22, 2026
1 check passed
@WalksWithASwagger WalksWithASwagger deleted the codex/issue-51-digest-approved-routing branch May 22, 2026 01:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-human Agentic automation stopped for human judgment.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add human-approved routing for resurfacing digests

1 participant