chore: drafting signal logs - #467
Conversation
…026-08-04 Co-Authored-By: Oz <oz-agent@warp.dev>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
…026-08-05 Co-Authored-By: Oz <oz-agent@warp.dev>
…026-08-06 Co-Authored-By: Oz <oz-agent@warp.dev>
|
I'm starting a first review of this pull request. You can view the conversation on Warp. I completed the review and no human review was requested for this pull request. Comment Powered by Oz |
There was a problem hiding this comment.
Overview
This PR updates the drafting-skill signal logs by appending parsed human review feedback records and adding collector-run summaries. The JSONL additions parse successfully, the changes are scoped to .agents/logs/, and there are no security findings or approved spec context to validate against.
Concerns
- Several new
pr_review_runs.mdcollector entries were inserted inside the fencedFormatexample instead of after the separator with the real run-log entries, so they render as template/example text rather than durable log entries.
Verdict
Found: 0 critical, 1 important, 0 suggestions
Request changes
Comment /warp-agent-review on this pull request to retrigger a review (up to 3 times on the same pull request).
Powered by Oz
| **Format**: | ||
| ```markdown | ||
|
|
||
| ## 2026-09-01 — collector scan (improve-drafting-skills monthly) |
There was a problem hiding this comment.
Format example; move the 2026-09-01, 2026-08-06, and 2026-08-04 sections below the --- separator so they are actual log entries.
Summary
Standing PR that accumulates drafting skill signal logs (
.agents/logs/human_review_feedback.jsonland.agents/logs/pr_review_runs.md) from the monthlyimprove-drafting-skillsouter loop.Merge periodically (at least before each monthly run) so logs reach
main. Do not mix skill/template edits into this branch.Test plan
.agents/logs/changed