@tea-agent/loop-agent 0.21.0 → 0.22.0

package/skills/frontend-verification/SKILL.md

@@ -17,18 +17,22 @@ verdict/findings, and required browser, visual, manual, or knowledge evidence.
 
 ## Evidence Rules
 
-- Static evidence covers type/lint/build/schema; behavior evidence must exercise the flow.
+- Static evidence covers type/build/schema; behavior evidence must exercise the flow.
+  Lint has its own assessment status: `passed`, `baseline-debt`, `failed`, or
+  `unavailable`.
 - Shell exit status is authoritative. Classify as `passed`, `failed`, `not-run`,
   `blocked`, or `unavailable`; only passed satisfies a required check.
 - Never use static success as behavior proof, or tests as visual/browser proof they did not exercise.
 - Mock-backed behavior proves frontend rendering and state transitions only. It never
   proves backend readiness, transport compatibility, or real API integration.
 - Unavailable commands remain gaps.
-- Resolve design evidence from two parallel sources: query the execution environment's
-  knowledge base connector when available; regardless of result, also read
-  `<repoRoot>/openspec/**` for index and task-relevant specification content.
-  The connector format remains TODO; never invent it. Applied `openspec` rules
-  from successful reads are available project evidence.
+- `baseline-debt` is not lint passed. It requires
+  `frontend-lint-assessment-v1` proving only pre-writer diagnostics on unchanged
+  files. Report debt count, changed files, and raw evidence. Typecheck/build/test do
+  not support debt.
+- Resolve design evidence from parallel sources: query the available knowledge
+  connector and always read task-relevant `<repoRoot>/openspec/schemas/`,
+  `<repoRoot>/openspec/project-specs/`, and `<repoRoot>/ai_workspace/`. Connector format is TODO; never invent it.
 - Separate Mock service/handler checks from page consumption and record the
   dev/test-only boundary; handler tests alone do not prove page use.
 
@@ -40,18 +44,17 @@ only proven changes.
 Return Markdown headings:
 
 - `Changes`: changed behavior and areas.
-- `Mock Decision`, `Mock Files`, `Mock Verification`, `Production Boundary`: status is `passed`, `failed`, `not-required`, `blocked`, or `unavailable`.
+- `Mock Decision`, `Mock Files`, `Mock Verification`, `Production Boundary`.
 - `Verification Evidence`: table of check, command/source, status, and artifact/result.
+  Keep lint `baseline-debt` verbatim rather than converting it to `passed`.
 - `Review Result`: exact review verdict and findings.
 - `Known Risks`: missing optional checks and environment caveats.
 - `Follow-up`: concrete work or `None`.
 
-When the backend remains unavailable but required mock-backed checks pass, state
-`Frontend status: mock-validated` and `Real integration: pending`. When default
-`auto` skipped Mock and no real API evidence passed, state
-`Frontend status: locally-validated` and `Real integration: pending`. Use a completed
-`<task-id>-real-api-integration-verify` task before changing the latter to complete;
-the follow-up is explicit, not auto-created or auto-executed.
+With only Mock evidence, state `Frontend status: mock-validated`; when default `auto`
+skipped Mock without real API evidence, state `Frontend status: locally-validated`.
+Both require `Real integration: pending` and explicit
+`<task-id>-real-api-integration-verify` follow-up; it is not auto-created/executed.
 
 Do not edit files. Do not claim complete when review is not pass or a required check
 is failed, not-run, blocked, unavailable, stale, or contradicted.

package/skills/frontend-verification/references/verification-checklist.md

@@ -3,19 +3,26 @@
 ## Static And Behavior Evidence
 
 - Required type/compile, lint/format, build, schema/client, browser/e2e/manual checks ran.
+- Lint is recorded as `passed | baseline-debt | failed | unavailable`.
+  `baseline-debt` cites `frontend-lint-baseline-v1` and
+  `frontend-lint-assessment-v1`, writer changed files, tolerated diagnostic count,
+  command identity, and raw output hashes; it is never labeled passed.
+- Every remaining lint diagnostic is on an unchanged file and matches the
+  writer-preceding baseline. Changed-file diagnostics, new unmatched diagnostics,
+  command drift, timeout, worktree mutation, or unparseable output fail closed.
+- Typecheck, build, and test are fully passed; none accepts baseline debt.
 - Generated output was authorized; tests cover changed logic, flows, and regressions.
-- Fixed DAG entrypoints prove selected-strategy behavior and applicable loading/empty/error/success/disabled/permission/retry/boundary states.
+- Fixed entrypoints prove selected-strategy behavior and applicable UI states.
 - Mock-specific checks cover service/handler/schema/fixtures; behavior evidence separately proves page consumption.
 - Mock activation is explicit/non-production; a default-real-path build with Mock off proves the real request remains default.
-- `not-needed` has real/no-remote evidence, or default-auto skipped-Mock rationale with Real Integration Gap preserved when no project Mock capability exists.
+- `not-needed` has real/no-remote evidence or a default-auto skipped-Mock rationale.
 - Mock-backed evidence is frontend-only and never satisfies real API integration.
 
 ## Design And Component Evidence
 
-- Claims cite two parallel sources: knowledge-base retrieval and
-  `<repoRoot>/openspec/**`.
-- Evidence records query/source/time for knowledge base plus openspec search terms, paths, headings, and applied rules for both.
-- Relevant `openspec/` matches satisfy source availability; missing both sources blocks explicit compliance or required design decisions.
+- Claims cite knowledge-base, OpenSpec, and `<repoRoot>/ai_workspace/` sources.
+- Evidence records query/source/time, local-spec terms, paths, headings, and rules.
+- OpenSpec or `ai_workspace/` matches satisfy source availability; none blocks required design decisions.
 
 ## Status
 
@@ -25,7 +32,8 @@
 - `blocked`: a prerequisite prevented execution.
 - `unavailable`: tool, environment, connector, or source was absent.
 
-Only passed satisfies a required check. Other optional statuses remain disclosed risks.
+Only passed satisfies a required check. For lint alone, evidence-backed
+`baseline-debt` may continue as explicit debt; it replaces no other check.
 
 ## Closeout Checks
 
@@ -35,26 +43,6 @@ Only passed satisfies a required check. Other optional statuses remain disclosed
 - If only Mock evidence exists, report `Frontend status: mock-validated` and `Real integration: pending`, with actual API verification follow-up.
 - If default `auto` skipped Mock and no real API evidence exists, report `Frontend status: locally-validated` and `Real integration: pending`.
 
-```markdown
-## Changes
-- ...
-
-## Verification Evidence
-| Check | Command or source | Status | Evidence |
-|---|---|---|---|
-| ... | ... | passed | ... |
-
-## Mock Decision / Mock Files / Mock Verification / Production Boundary
-- Status: `passed | failed | not-required | blocked | unavailable`
-
-## Review Result
-- Verdict: `VERDICT: pass`
-
-## Known Risks
-- ...
-
-## Follow-up
-- None.
-```
-
-If review is not pass or a required check is not passed, describe the task as incomplete and list concrete follow-up.
+Output uses Changes, Verification Evidence, Mock Decision/Files/Verification,
+Production Boundary, Review Result, Known Risks, and Follow-up headings. If review is
+not pass or a required check fails, describe the task as incomplete with follow-up.