✨ Add per-prompt Rudder test reports - #94
Merged
Merged
Conversation
Co-authored-by: Codex Agent <codex@openai.com>
📦 No plugin release on mergeThe plugin package, tag, and GitHub Release already exist for |
|
Ready to review this PR? Stage has broken it down into 4 individual chapters for you:
Chapters generated by Stage for commit c011c1a on Jul 30, 2026 5:57pm UTC. |
| session_id AS sessionId, | ||
| prompt_id AS promptId, | ||
| prompt_text AS promptText, | ||
| previous_agent_output AS previousAgentOutput, |
There was a problem hiding this comment.
When an existing Rudder database predates the previous_agent_output migration and has not subsequently been migrated, the read-only context helper unconditionally selects that column, causing SQLite to terminate the helper without emitting the context JSON required to generate the report.
| Use `completed` when the workflow reaches its normal report, `stopped` when it ends by user choice or missing intent, and `blocked` only for an external blocker. | ||
| Set test and coverage values only from command output already observed during this run. | ||
| Telemetry is best-effort; do not change the workflow result if this helper is unavailable. | ||
| 13. After the generation loop ends, create and present the prompt report. |
There was a problem hiding this comment.
Comment on lines
+280
to
+283
| Use `completed` when the workflow reaches its normal report, `stopped` when it ends by user choice or missing intent, and `blocked` only for an external blocker. | ||
| Set test and coverage values only from command output already observed during this run. | ||
| Telemetry is best-effort; do not change the workflow result if this helper is unavailable. | ||
| 13. After the generation loop ends, create and present the prompt report. |
There was a problem hiding this comment.
Co-authored-by: Codex Agent <codex@openai.com>
Co-authored-by: Codex Agent <codex@openai.com>
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.
Adds a temporary Markdown report at the end of each Rudder test-generation run, grouping human-readable test links by their captured prompts. Prompt groups include exact prompt text and a concise representation of prior agent context from the database. The context helper now exposes previous agent output, with runtime coverage for that behavioral field. Validation passed with npm run typecheck, npm test, and npm run build.
Greptile Summary
Adds a per-prompt Markdown report to Rudder test-generation runs.
previousAgentOutputfrom a captured transcript.Confidence Score: 4/5
The PR should not merge until legacy database reads, early-run report finalization, and completion telemetry ordering are corrected.
The helper can terminate on pre-migration state, early workflow exits omit a report required for every run, and telemetry can mark a run complete before required report generation succeeds.
Files Needing Attention: skills/rudder/scripts/context.mjs and skills/rudder/SKILL.md
Important Files Changed
Sequence Diagram
sequenceDiagram participant R as Rudder workflow participant C as context.mjs participant D as Prompt database participant T as Telemetry participant F as Temporary report R->>C: Refresh run context C->>D: Read promptText and previousAgentOutput D-->>C: Captured prompt records C-->>R: Context JSON R->>R: Match final tagged tests to prompts R->>T: Record completion R->>F: Create Markdown report F-->>R: Temporary report path R-->>R: Present report first in final responseReviews (1): Last reviewed commit: "✨ Add per-prompt Rudder test reports" | Re-trigger Greptile