Add workflow event inspection for agent diagnostics - #24
Conversation
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (17)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@codex review |
|
Codex Review: Didn't find any major issues. Another round soon, please! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
226e4b8 to
1e71dfe
Compare
1e71dfe to
f448cc7
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f448cc76c3
ℹ️ 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".
| "@types/node": "26.2.0", | ||
| "qyl-mcp-dashboard": "^1.1.1", | ||
| "qyl-mcp-server": "^1.1.1", | ||
| "qyl-mcp-server": "^1.2.0", |
There was a problem hiding this comment.
Regenerate the Bun lockfile with the contract bump
In environments that install with the repository-declared Bun package manager, bun.lock still records qyl-mcp-server 1.1.x and @ancplua/qyl-api-schema 7.0.0 for every workspace, even though these manifests now require 1.2.0/7.1.0. This makes the committed dependency graph non-reproducible and causes frozen Bun installs to reject it—bun install --help defines --frozen-lockfile as “Disallow changes to lockfile”—rather than installing the generated contracts needed by the new tool. Regenerate and commit bun.lock alongside package-lock.json.
AGENTS.md reference: AGENTS.md:L7-L7
Useful? React with 👍 / 👎.
Summary
inspect_workflow_eventstool for immutable workflow journal inspection.InspectWorkflowEventsInput/Outputcontract from@ancplua/qyl-api-schema@7.1.0: immediate bounded reads with no graph cursors or long-poll controls.content_capturedsummaries first and retrieve protected evidence only through an explicit listedcontent_ref.qyl-mcp-server1.2.0, align workspace consumers, and regenerate the tool manifest at contract revisionsha256:72545708596baccd.Stack
510a7115.Validation
npm cinpm run buildnpm test --workspace server— 68/68 passed.npm run smoke --workspace servernpm run verify:pinsagainst merged qyl main — 7.1.0 pins agree.npm run smoke:otlpagainst merged qyl main — official OTLP persistence, exact trace evidence, redaction, and the livecontent_capturedsummary → explicit protected-content round trip passed in 18 seconds.