Skip to content

[skills-daily] keep coverage command mapping current#104

Merged
david30907d merged 1 commit into
mainfrom
docs/agent-skills-ongoing-20260709
Jul 12, 2026
Merged

[skills-daily] keep coverage command mapping current#104
david30907d merged 1 commit into
mainfrom
docs/agent-skills-ongoing-20260709

Conversation

@taiiiiiiiiiiii

Copy link
Copy Markdown
Contributor

Summary

  • Verified today's merged skill cleanup work: .agents/AGENTS.md exists and monorepo CI guidance has already been compacted/split.
  • Fixed one small drift introduced by the compaction: the monorepo CI coverage row now includes the workflow's pnpm run coverage test self-test before workspace coverage summary.
  • Added the same self-test command to the handoff verification block.

Inspected

Decision log

  • No new skill: the requested AGENTS.md and monorepo split are already merged.
  • Update existing skill/doc: .agents/skills/monorepo-ci-debugging/SKILL.md because it maps CI jobs to local commands and had one stale coverage command.
  • No further cleanup: existing split skills are within the new .agents/AGENTS.md size/quality budget.

Validation

  • Docs-only diff: one skill file, +2/-1 lines.
  • Workflow command checked against .github/workflows/ci.yml.

Auto-merge

  • Not enabled at PR creation. Wait for GitHub checks/mergeability; repository auto-merge availability must remain the source of truth.

@vercel

vercel Bot commented Jul 9, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
zap-engine-frontend Ready Ready Preview, Comment Jul 9, 2026 12:58pm
zap-engine-landing-page Ready Ready Preview, Comment Jul 9, 2026 12:58pm

Copy link
Copy Markdown
Contributor Author

Run summary:

Copy link
Copy Markdown
Contributor Author

Run summary (2026-07-10 JST):

  • Inspected the active skills/docs PR [skills-daily] keep coverage command mapping current #104, its one-file patch, current labels/mergeability, CI run #625, recent commits since the previous run, and the Friday skill-authoring guardrails in .agents/AGENTS.md.
  • Continued [skills-daily] keep coverage command mapping current #104; no second overlapping PR was opened.
  • No additional doc change: the new WalletConnect/Privy work exposed one Metro-specific wagmi/connectors barrel-import issue, but it has appeared only once and is already enforced in code through import restrictions and subpath imports. It does not yet meet the repeated-failure bar for a new or expanded skill.
  • Friday bloat audit: no further compaction was warranted. The broad monorepo CI guidance was already split in docs: compact monorepo CI skills #103, and .agents/AGENTS.md now enforces 60–120 line targets, a 180-line maximum, and scheduled-task anti-append rules.
  • PR [skills-daily] keep coverage command mapping current #104 remains docs-only and mergeable, and the agent-skills label is present.
  • Auto-merge was not enabled: GitHub Actions run #625 completed with lint-test failing at Run check-variant tasks; coverage, dead-env, deploy gates, and both Vercel checks passed. Required checks are therefore not green.
  • Watch next: whether Metro/Expo repeatedly breaks on broad web-only package barrels (wagmi/connectors or similar). Only after a second occurrence should the existing app/web boundary guidance be updated with the exact subpath-import rule.

@david30907d david30907d merged commit 4cc6610 into main Jul 12, 2026
8 of 9 checks passed
@david30907d david30907d deleted the docs/agent-skills-ongoing-20260709 branch July 12, 2026 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants