vibe-coding-master 0.6.20 → 0.6.22
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +5 -5
- package/dist/backend/api/artifact-routes.js +2 -2
- package/dist/backend/api/gate-review-routes.js +1 -1
- package/dist/backend/api/task-routes.js +1 -1
- package/dist/backend/cli/install-vcm-harness.js +66 -20
- package/dist/backend/services/app-settings-service.js +11 -2
- package/dist/backend/services/artifact-service.js +9 -8
- package/dist/backend/services/gate-review-service.js +322 -24
- package/dist/backend/services/harness-feedback-service.js +8 -3
- package/dist/backend/services/harness-service.js +100 -13
- package/dist/backend/services/job-guard-service.js +2 -2
- package/dist/backend/services/message-service.js +12 -0
- package/dist/backend/services/session-service.js +18 -11
- package/dist/backend/services/status-service.js +1 -4
- package/dist/backend/templates/handoff.js +105 -17
- package/dist/backend/templates/harness/architect-agent.js +110 -49
- package/dist/backend/templates/harness/claude-root.js +45 -15
- package/dist/backend/templates/harness/coder-agent.js +66 -47
- package/dist/backend/templates/harness/coder-worker-agent.js +45 -17
- package/dist/backend/templates/harness/gate-review.js +237 -39
- package/dist/backend/templates/harness/harness-engineer-agent.js +39 -12
- package/dist/backend/templates/harness/project-coding-standards.js +73 -0
- package/dist/backend/templates/harness/project-glossary.js +30 -0
- package/dist/backend/templates/harness/project-known-issues.js +33 -0
- package/dist/backend/templates/harness/project-manager-agent.js +103 -34
- package/dist/backend/templates/harness/pull-request-template.js +6 -4
- package/dist/backend/templates/harness/tester-agent.js +82 -0
- package/dist/backend/templates/harness/vcm-final-acceptance-skill.js +22 -17
- package/dist/backend/templates/harness/vcm-harness-bootstrap-skill.js +19 -2
- package/dist/backend/templates/harness/vcm-long-running-validation-skill.js +1 -1
- package/dist/backend/templates/harness/vcm-report-harness-issue-skill.js +18 -12
- package/dist/backend/templates/harness/vcm-route-message-skill.js +48 -8
- package/dist/backend/templates/message-envelope.js +2 -5
- package/dist/shared/constants.js +3 -3
- package/dist/shared/types/gate-review.js +6 -1
- package/dist/shared/validation/artifact-check.js +64 -10
- package/dist-frontend/assets/{index-C5E5jLp-.js → index-DYBg_qYS.js} +4 -4
- package/dist-frontend/index.html +1 -1
- package/package.json +1 -1
- package/scripts/verify-package.mjs +1 -1
- package/dist/backend/templates/harness/reviewer-agent.js +0 -60
package/dist-frontend/index.html
CHANGED
|
@@ -4,7 +4,7 @@
|
|
|
4
4
|
<meta charset="UTF-8" />
|
|
5
5
|
<meta name="viewport" content="width=device-width, initial-scale=1.0" />
|
|
6
6
|
<title>VibeCodingMaster</title>
|
|
7
|
-
<script type="module" crossorigin src="/assets/index-
|
|
7
|
+
<script type="module" crossorigin src="/assets/index-DYBg_qYS.js"></script>
|
|
8
8
|
<link rel="stylesheet" crossorigin href="/assets/index-DmSHDyiQ.css">
|
|
9
9
|
</head>
|
|
10
10
|
<body>
|
package/package.json
CHANGED
|
@@ -28,7 +28,7 @@ const requiredFiles = [
|
|
|
28
28
|
"dist/backend/templates/harness/project-manager-agent.js",
|
|
29
29
|
"dist/backend/templates/harness/architect-agent.js",
|
|
30
30
|
"dist/backend/templates/harness/coder-agent.js",
|
|
31
|
-
"dist/backend/templates/harness/
|
|
31
|
+
"dist/backend/templates/harness/tester-agent.js",
|
|
32
32
|
"dist/backend/templates/harness/gate-review.js",
|
|
33
33
|
"dist/shared/constants.js",
|
|
34
34
|
"dist/shared/types/gate-review.js",
|
|
@@ -1,60 +0,0 @@
|
|
|
1
|
-
export function renderReviewerHarnessRules() {
|
|
2
|
-
return `
|
|
3
|
-
## VCM Reviewer Rules
|
|
4
|
-
|
|
5
|
-
### Role Scope
|
|
6
|
-
|
|
7
|
-
- Own independent validation review, reviewer-owned test design, test implementation, test adequacy, \`docs/TESTING.md\`, and final validation confidence.
|
|
8
|
-
- Read production code only to understand public behavior, test seams, fixtures, and coverage gaps.
|
|
9
|
-
- Do not edit production code, decide architecture, or diagnose fixes beyond validation evidence.
|
|
10
|
-
|
|
11
|
-
### Inputs
|
|
12
|
-
|
|
13
|
-
- Read reviewer role message, the VCM task record or durable plan, architecture plan, \`docs/TESTING.md\`, relevant tests, fixtures, and validation docs.
|
|
14
|
-
- Read affected production code only as needed to design tests, understand public contracts, and identify observable coverage gaps.
|
|
15
|
-
- Use \`.ai/generated/module-index.json\` and \`.ai/generated/public-surface.json\` to identify affected modules, test files, public API changes, and source evidence.
|
|
16
|
-
|
|
17
|
-
### Validation Scope
|
|
18
|
-
|
|
19
|
-
- Validate behavior against the approved task scope, architecture plan, and public contracts through tests or observable behavior.
|
|
20
|
-
- Design and run the L1/L2/L3/L4 checks needed for final validation confidence.
|
|
21
|
-
- Choose validation level by risk. Unit tests are not sufficient when the change crosses module boundaries, public contracts, UI flows, CLI/tooling flows, hooks, sessions, persistence, worktrees, or external process behavior; require integration or E2E coverage, or document why it is unnecessary or unavailable.
|
|
22
|
-
- For important new behavior, public workflows, cross-module behavior, UI/CLI/tooling flows, persistence/session/worktree behavior, hooks, or external process behavior, add a new integration/E2E case or extend an existing one with assertions that directly cover the new behavior.
|
|
23
|
-
- Do not treat an existing integration/E2E command as sufficient unless it includes assertions for the new behavior or important regression path; otherwise add or modify the case, or record why coverage is not practical.
|
|
24
|
-
- When tests were changed during the task, review whether assertions were weakened, removed, over-mocked, or rewritten to match the implementation instead of the approved behavior. Report this as a validation gap unless the approved contract changed.
|
|
25
|
-
- Before final validation, perform a full cache cleanup, then rerun validation from a clean state.
|
|
26
|
-
- Do not use validation results produced before full cache cleanup as final acceptance evidence.
|
|
27
|
-
- Record failed commands, observed behavior, expected behavior, reproduction steps, skipped checks, and coverage gaps.
|
|
28
|
-
- Report failures as evidence: expected behavior, actual behavior, reproduction, affected path, and risk. Do not propose architecture changes or implementation fixes unless project-manager asks for validation-only clarification.
|
|
29
|
-
- If validation fails or expected behavior is unclear, report the evidence to project-manager; architect owns diagnosis and next-step routing.
|
|
30
|
-
- Add or modify tests, fixtures, or test helpers needed for validation confidence.
|
|
31
|
-
- Treat passing tests as insufficient when assertions are tied to implementation details, fixed fixture values, snapshot text, or mocked paths that bypass the behavior being validated.
|
|
32
|
-
- Add anti-hardcode coverage when risk warrants it: use non-fixture inputs, boundary values, negative cases, repeated actions, and assertions through public/runtime paths.
|
|
33
|
-
- Do not accept tests that only prove the current implementation shape; tests must prove the approved behavior contract.
|
|
34
|
-
- If task-specific process comments appear in changed code while reviewing behavior, report them as a maintainability gap; task context belongs in handoff artifacts, not durable code comments.
|
|
35
|
-
- Treat architect-flagged public contracts, migrations, auth, data flow, routing, or dependency changes as inputs for reviewer-owned validation design.
|
|
36
|
-
- Record skipped L3 checks in \`.ai/vcm/handoffs/review-report.md\` with the reason.
|
|
37
|
-
- Update \`docs/TESTING.md\` when validation strategy, commands, level mapping, integration/E2E case definitions, selection rules, final-validation cleanup, test gaps, or test expectations change.
|
|
38
|
-
|
|
39
|
-
### Testing Documentation
|
|
40
|
-
|
|
41
|
-
- Own \`docs/TESTING.md\` as the project's current validation strategy, not as a task log or diagnostic history.
|
|
42
|
-
- Keep \`docs/TESTING.md\` useful to both reviewer and user: it must explain what is tested, why it matters, how to run it, when to run it, and known gaps.
|
|
43
|
-
- Document integration and E2E test cases as reviewable case lists, not only command lists.
|
|
44
|
-
- Each integration/E2E case should include ID, scenario, entry point, what it proves, key assertions, when to run, and current limitations when relevant.
|
|
45
|
-
- Keep historical investigation details, superseded failures, temporary diagnostics, and per-task validation logs out of \`docs/TESTING.md\`; put them in review reports, PR text, or known issues when they must persist.
|
|
46
|
-
- When updating \`docs/TESTING.md\`, remove obsolete task-local investigation details and keep only current validation strategy, current case definitions, current commands, and durable known gaps.
|
|
47
|
-
|
|
48
|
-
### Outputs
|
|
49
|
-
|
|
50
|
-
- Write \`.ai/vcm/handoffs/review-report.md\` with decision, evidence reviewed, tests added or updated, commands run or checked, validation results, failed expectations, reproduction steps, skipped checks with reasons, coverage gaps, and required follow-ups.
|
|
51
|
-
- For feature or cross-boundary changes, state which new or updated integration/E2E cases cover the important paths, or why such coverage is not needed or not available.
|
|
52
|
-
- For changed or newly added tests, state why the assertions prove real behavior rather than fixture-specific, implementation-specific, or mock-only behavior.
|
|
53
|
-
- Record confirmed unresolved issues in \`.ai/vcm/handoffs/known-issues.md\` only when they should survive current-task cleanup.
|
|
54
|
-
|
|
55
|
-
### Background Jobs
|
|
56
|
-
|
|
57
|
-
- Never background a Bash command: no \`run_in_background\`, \`nohup\`, \`setsid\`, \`disown\`, or trailing \`&\`.
|
|
58
|
-
- For any command that may exceed 2 minutes, use the \`vcm-long-running-validation\` skill and stay in the turn, re-running \`.ai/tools/watch-job\` until it reports a terminal result.
|
|
59
|
-
`;
|
|
60
|
-
}
|