docs(skills): make aeo_crosslink_audit reliable#280
Conversation
Assessment of the scheduled AEO cross-link agent found the run log has never populated and several noise sources in the automated PRs. Fixes: - Run logging: never commit to protected main (direct commits failed silently, leaving the log empty and blocking improve-aeo-crosslink-skill). Use a standing chore/aeo-crosslink-audit-log branch + one perpetual PR. - Dedup/supersede: check for an existing open cross-link PR before opening; never leave two open at once (prevents duplicates like #232 vs #228). - Low-signal runs: prefer a no-change report over weak docs-only PRs when neither Peec nor GSC signals are usable. - Restructure-safety: re-validate edited files and link targets against the latest main and resolve redirects before opening (prevents #262/#265-style breakage where cloud-agents moved to platform/). - Document biweekly cadence. - improve-skill: note the standing log PR must be merged before analysis. Co-Authored-By: Oz <oz-agent@warp.dev>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This comment was marked as outdated.
This comment was marked as outdated.
…time Co-Authored-By: Oz <oz-agent@warp.dev>
![oz-for-oss[bot]](https://avatars.githubusercontent.com/in/3450139?s=60&v=4){kind=small}
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
Co-Authored-By: Oz <oz-agent@warp.dev>
rachaelrenk
changed the title
|
/oz-review |
|
I'm re-reviewing this pull request in response to a review 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 hardens the AEO cross-link audit skills by adding low-signal gating, duplicate-PR checks, fresh-tree validation, redirect checks, and a standing log-PR workflow.
Concerns
- The fresh-tree validation command can leave
origin/mainstale while the new workflow explicitly tells agents to validate edited files and link targets againstorigin/main. That undermines the restructure-safety this PR is adding.
Verdict
Found: 0 critical, 1 important, 0 suggestions
Request changes
Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).
Powered by Oz
Co-authored-by: oz-for-oss[bot] <277970191+oz-for-oss[bot]@users.noreply.github.com>
2 participants
Summary
Assessment of the scheduled
aeo_crosslink_auditagent (11 PRs, Jun 3–29) found that output quality is fine, but the workflow has real reliability gaps and produces avoidable automated-PR noise. This PR hardens the skill. The schedule stays weekly — no cadence change.Key findings:
.agents/logs/aeo_crosslink_audit_runs.mdhas zero entries despite run logging being added in docs: add run logging and Slack notifications to aeo_crosslink_audit skill #234. Root cause: step 7 committed the log directly tomain, which is a protected branch, so the push failed and the silent fallback hid every entry in Oz run output. This also blocksimprove-aeo-crosslink-skill(needs 8 log entries).agent-platform/cloud-agents/*to the new top-levelplatform/*section; the now-closed docs: add AEO cross-links for agents and orchestration #265 edited a moved file and linked to stale/agent-platform/cloud-agents/*routes (now redirects).Changes
.agents/skills/aeo_crosslink_audit/SKILL.md
main. Each run records its entry through a standingchore/aeo-crosslink-audit-logbranch and one perpetual, low-noise PR that accumulates every run's entry regardless of outcome.origin/mainand re-validate every edited file and link target before opening; resolvevercel.jsonredirects to their final destinations..agents/skills/improve-aeo-crosslink-skill/SKILL.md
main(or read from its branch) before monthly analysis.Notes for reviewers
0 15 * * 1, Mondays 15:00 UTC).platform/.Conversation: https://staging.warp.dev/conversation/2ed9c801-c150-4cfc-994a-4ea18972af46
Co-Authored-By: Oz oz-agent@warp.dev