Skip to content

fix: unify runner egress and AOP persistence - #115

Open
M09Ic wants to merge 2 commits into
masterfrom
feat/mitm-stream-capture
Open

fix: unify runner egress and AOP persistence#115
M09Ic wants to merge 2 commits into
masterfrom
feat/mitm-stream-capture

Conversation

@M09Ic

@M09Ic M09Ic commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Summary

  • route built-in tools through runner egress consistently
  • make node/runtime AOP delivery single-source and idempotent by event ID
  • persist continuation history by explicit inheritance/snapshot metadata instead of replaying full messages and tool results
  • enforce one SQLite v1 schema and remove runtime legacy/compat migration paths
  • avoid duplicate scan reports and remove the frontend IOA compat adapter

Validation

  • 4 MiB continuation regression: inherited history is not re-emitted
  • DeepSeek real request: 80 JSONL events, ~19.9 KiB, zero duplicate event IDs
  • go test ./core/output ./pkg/web/service ./pkg/runner ./agent ./pkg/node -count=1

pm run build

  • go test ./... -count=1 (all packages pass except existing environment-dependent ools/katana/TestE2EHeadlessReusesDiscoveredBrowser)

Note

  • The branch is based on current master but intentionally removes the upstream JSONL/SQLite hard caps from the final diff; those caps silently discard events, contrary to the no-data-loss requirement. The fix is idempotent persistence and single-source emission.

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