llm-mock-server 1.0.6 → 1.0.7
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/dist/cli/cli.d.ts +3 -0
- package/dist/cli/cli.d.ts.map +1 -0
- package/dist/cli/cli.js +103 -0
- package/dist/cli/cli.js.map +1 -0
- package/dist/cli/validators.d.ts +7 -0
- package/dist/cli/validators.d.ts.map +1 -0
- package/dist/cli/validators.js +53 -0
- package/dist/cli/validators.js.map +1 -0
- package/dist/formats/anthropic/index.d.ts +1 -1
- package/dist/formats/anthropic/index.d.ts.map +1 -1
- package/dist/formats/anthropic/index.js +1 -1
- package/dist/formats/anthropic/index.js.map +1 -1
- package/dist/formats/anthropic/parse.d.ts +2 -2
- package/dist/formats/anthropic/parse.d.ts.map +1 -1
- package/dist/formats/anthropic/parse.js +4 -2
- package/dist/formats/anthropic/parse.js.map +1 -1
- package/dist/formats/anthropic/schema.d.ts +1 -1
- package/dist/formats/anthropic/schema.d.ts.map +1 -1
- package/dist/formats/anthropic/schema.js +9 -4
- package/dist/formats/anthropic/schema.js.map +1 -1
- package/dist/formats/anthropic/serialize.d.ts +2 -2
- package/dist/formats/anthropic/serialize.d.ts.map +1 -1
- package/dist/formats/anthropic/serialize.js +76 -19
- package/dist/formats/anthropic/serialize.js.map +1 -1
- package/dist/formats/openai/chat-completions/index.d.ts +3 -0
- package/dist/formats/openai/chat-completions/index.d.ts.map +1 -0
- package/dist/formats/openai/chat-completions/index.js +13 -0
- package/dist/formats/openai/chat-completions/index.js.map +1 -0
- package/dist/formats/openai/chat-completions/parse.d.ts +4 -0
- package/dist/formats/openai/chat-completions/parse.d.ts.map +1 -0
- package/dist/formats/openai/chat-completions/parse.js +33 -0
- package/dist/formats/openai/chat-completions/parse.js.map +1 -0
- package/dist/formats/openai/chat-completions/schema.d.ts +93 -0
- package/dist/formats/openai/chat-completions/schema.d.ts.map +1 -0
- package/dist/formats/openai/chat-completions/schema.js +74 -0
- package/dist/formats/openai/chat-completions/schema.js.map +1 -0
- package/dist/formats/openai/chat-completions/serialize.d.ts +10 -0
- package/dist/formats/openai/chat-completions/serialize.d.ts.map +1 -0
- package/dist/formats/openai/chat-completions/serialize.js +99 -0
- package/dist/formats/openai/chat-completions/serialize.js.map +1 -0
- package/dist/formats/openai/responses/index.d.ts +3 -0
- package/dist/formats/openai/responses/index.d.ts.map +1 -0
- package/dist/formats/openai/responses/index.js +13 -0
- package/dist/formats/openai/responses/index.js.map +1 -0
- package/dist/formats/openai/responses/parse.d.ts +4 -0
- package/dist/formats/openai/responses/parse.d.ts.map +1 -0
- package/dist/formats/openai/responses/parse.js +51 -0
- package/dist/formats/openai/responses/parse.js.map +1 -0
- package/dist/formats/openai/responses/schema.d.ts +103 -0
- package/dist/formats/openai/responses/schema.d.ts.map +1 -0
- package/dist/formats/openai/responses/schema.js +71 -0
- package/dist/formats/openai/responses/schema.js.map +1 -0
- package/dist/formats/openai/responses/serialize.d.ts +10 -0
- package/dist/formats/openai/responses/serialize.d.ts.map +1 -0
- package/dist/formats/openai/responses/serialize.js +273 -0
- package/dist/formats/openai/responses/serialize.js.map +1 -0
- package/dist/formats/request-helpers.d.ts +1 -1
- package/dist/formats/request-helpers.d.ts.map +1 -1
- package/dist/formats/request-helpers.js.map +1 -1
- package/dist/formats/serialize-helpers.d.ts +1 -1
- package/dist/formats/serialize-helpers.d.ts.map +1 -1
- package/dist/formats/serialize-helpers.js +6 -3
- package/dist/formats/serialize-helpers.js.map +1 -1
- package/dist/formats/types.d.ts +2 -1
- package/dist/formats/types.d.ts.map +1 -1
- package/dist/history.d.ts +6 -2
- package/dist/history.d.ts.map +1 -1
- package/dist/history.js +2 -0
- package/dist/history.js.map +1 -1
- package/dist/index.d.ts.map +1 -1
- package/dist/index.js.map +1 -1
- package/dist/loader.d.ts +1 -1
- package/dist/loader.d.ts.map +1 -1
- package/dist/loader.js +26 -9
- package/dist/loader.js.map +1 -1
- package/dist/logger.d.ts.map +1 -1
- package/dist/logger.js +12 -4
- package/dist/logger.js.map +1 -1
- package/dist/mock-server.d.ts +44 -48
- package/dist/mock-server.d.ts.map +1 -1
- package/dist/mock-server.js +37 -85
- package/dist/mock-server.js.map +1 -1
- package/dist/route-handler.d.ts +1 -1
- package/dist/route-handler.d.ts.map +1 -1
- package/dist/route-handler.js +19 -7
- package/dist/route-handler.js.map +1 -1
- package/dist/rule-builder.d.ts +21 -0
- package/dist/rule-builder.d.ts.map +1 -0
- package/dist/rule-builder.js +58 -0
- package/dist/rule-builder.js.map +1 -0
- package/dist/rule-engine.d.ts +3 -1
- package/dist/rule-engine.d.ts.map +1 -1
- package/dist/rule-engine.js +7 -2
- package/dist/rule-engine.js.map +1 -1
- package/dist/sse-writer.d.ts +1 -1
- package/dist/sse-writer.d.ts.map +1 -1
- package/dist/types/reply.d.ts +51 -8
- package/dist/types/reply.d.ts.map +1 -1
- package/dist/types/request.d.ts +21 -6
- package/dist/types/request.d.ts.map +1 -1
- package/dist/types/rule.d.ts +65 -7
- package/dist/types/rule.d.ts.map +1 -1
- package/dist/types.d.ts +3 -3
- package/dist/types.d.ts.map +1 -1
- package/package.json +15 -9
- package/.claude/skills/desloppify/SKILL.md +0 -308
- package/.desloppify/external_review_sessions/ext_20260315_000339_a6cdc3e6/canonical_import_20260315_000801.json +0 -242
- package/.desloppify/external_review_sessions/ext_20260315_000339_a6cdc3e6/canonical_import_20260315_000905.json +0 -248
- package/.desloppify/external_review_sessions/ext_20260315_000339_a6cdc3e6/canonical_import_20260315_000917.json +0 -248
- package/.desloppify/external_review_sessions/ext_20260315_000339_a6cdc3e6/canonical_import_20260315_000950.json +0 -311
- package/.desloppify/external_review_sessions/ext_20260315_000339_a6cdc3e6/claude_launch_prompt.md +0 -17
- package/.desloppify/external_review_sessions/ext_20260315_000339_a6cdc3e6/review_result.json +0 -255
- package/.desloppify/external_review_sessions/ext_20260315_000339_a6cdc3e6/review_result.template.json +0 -22
- package/.desloppify/external_review_sessions/ext_20260315_000339_a6cdc3e6/reviewer_instructions.md +0 -20
- package/.desloppify/external_review_sessions/ext_20260315_000339_a6cdc3e6/session.json +0 -20
- package/.desloppify/external_review_sessions/ext_20260315_045546_0587ea3b/canonical_import_20260315_050000.json +0 -286
- package/.desloppify/external_review_sessions/ext_20260315_045546_0587ea3b/canonical_import_20260315_050028.json +0 -303
- package/.desloppify/external_review_sessions/ext_20260315_045546_0587ea3b/claude_launch_prompt.md +0 -17
- package/.desloppify/external_review_sessions/ext_20260315_045546_0587ea3b/review_result.json +0 -297
- package/.desloppify/external_review_sessions/ext_20260315_045546_0587ea3b/review_result.template.json +0 -22
- package/.desloppify/external_review_sessions/ext_20260315_045546_0587ea3b/reviewer_instructions.md +0 -20
- package/.desloppify/external_review_sessions/ext_20260315_045546_0587ea3b/session.json +0 -20
- package/.desloppify/query.json +0 -1312
- package/.desloppify/review_packet_blind.json +0 -1249
- package/.desloppify/review_packets/holistic_packet_20260315_000339.json +0 -1471
- package/.desloppify/review_packets/holistic_packet_20260315_045546.json +0 -1480
- package/.desloppify/review_packets/holistic_packet_20260315_185401.json +0 -1407
- package/.desloppify/review_packets/holistic_packet_20260315_185613.json +0 -1407
- package/.desloppify/state-typescript.json +0 -8438
- package/.desloppify/state-typescript.json.bak +0 -8432
- package/.desloppify/subagents/runs/20260315_185401/logs/batch-1.log +0 -384
- package/.desloppify/subagents/runs/20260315_185401/logs/batch-10.log +0 -484
- package/.desloppify/subagents/runs/20260315_185401/logs/batch-2.log +0 -408
- package/.desloppify/subagents/runs/20260315_185401/logs/batch-3.log +0 -416
- package/.desloppify/subagents/runs/20260315_185401/logs/batch-4.log +0 -360
- package/.desloppify/subagents/runs/20260315_185401/logs/batch-5.log +0 -360
- package/.desloppify/subagents/runs/20260315_185401/logs/batch-6.log +0 -364
- package/.desloppify/subagents/runs/20260315_185401/logs/batch-7.log +0 -428
- package/.desloppify/subagents/runs/20260315_185401/logs/batch-8.log +0 -388
- package/.desloppify/subagents/runs/20260315_185401/logs/batch-9.log +0 -500
- package/.desloppify/subagents/runs/20260315_185401/prompts/batch-1.md +0 -83
- package/.desloppify/subagents/runs/20260315_185401/prompts/batch-10.md +0 -108
- package/.desloppify/subagents/runs/20260315_185401/prompts/batch-2.md +0 -89
- package/.desloppify/subagents/runs/20260315_185401/prompts/batch-3.md +0 -91
- package/.desloppify/subagents/runs/20260315_185401/prompts/batch-4.md +0 -77
- package/.desloppify/subagents/runs/20260315_185401/prompts/batch-5.md +0 -77
- package/.desloppify/subagents/runs/20260315_185401/prompts/batch-6.md +0 -78
- package/.desloppify/subagents/runs/20260315_185401/prompts/batch-7.md +0 -94
- package/.desloppify/subagents/runs/20260315_185401/prompts/batch-8.md +0 -84
- package/.desloppify/subagents/runs/20260315_185401/prompts/batch-9.md +0 -112
- package/.desloppify/subagents/runs/20260315_185401/results/batch-1.raw.txt +0 -0
- package/.desloppify/subagents/runs/20260315_185401/results/batch-10.raw.txt +0 -0
- package/.desloppify/subagents/runs/20260315_185401/results/batch-2.raw.txt +0 -0
- package/.desloppify/subagents/runs/20260315_185401/results/batch-3.raw.txt +0 -0
- package/.desloppify/subagents/runs/20260315_185401/results/batch-4.raw.txt +0 -0
- package/.desloppify/subagents/runs/20260315_185401/results/batch-5.raw.txt +0 -0
- package/.desloppify/subagents/runs/20260315_185401/results/batch-6.raw.txt +0 -0
- package/.desloppify/subagents/runs/20260315_185401/results/batch-7.raw.txt +0 -0
- package/.desloppify/subagents/runs/20260315_185401/results/batch-8.raw.txt +0 -0
- package/.desloppify/subagents/runs/20260315_185401/results/batch-9.raw.txt +0 -0
- package/.desloppify/subagents/runs/20260315_185401/run.log +0 -36
- package/.desloppify/subagents/runs/20260315_185401/run_summary.json +0 -156
- package/.desloppify/subagents/runs/20260315_185613/holistic_findings_merged.json +0 -741
- package/.desloppify/subagents/runs/20260315_185613/logs/batch-1.log +0 -579
- package/.desloppify/subagents/runs/20260315_185613/logs/batch-10.log +0 -1537
- package/.desloppify/subagents/runs/20260315_185613/logs/batch-2.log +0 -829
- package/.desloppify/subagents/runs/20260315_185613/logs/batch-3.log +0 -927
- package/.desloppify/subagents/runs/20260315_185613/logs/batch-4.log +0 -429
- package/.desloppify/subagents/runs/20260315_185613/logs/batch-5.log +0 -276
- package/.desloppify/subagents/runs/20260315_185613/logs/batch-6.log +0 -450
- package/.desloppify/subagents/runs/20260315_185613/logs/batch-7.log +0 -730
- package/.desloppify/subagents/runs/20260315_185613/logs/batch-8.log +0 -698
- package/.desloppify/subagents/runs/20260315_185613/logs/batch-9.log +0 -938
- package/.desloppify/subagents/runs/20260315_185613/prompts/batch-1.md +0 -83
- package/.desloppify/subagents/runs/20260315_185613/prompts/batch-10.md +0 -108
- package/.desloppify/subagents/runs/20260315_185613/prompts/batch-2.md +0 -89
- package/.desloppify/subagents/runs/20260315_185613/prompts/batch-3.md +0 -91
- package/.desloppify/subagents/runs/20260315_185613/prompts/batch-4.md +0 -77
- package/.desloppify/subagents/runs/20260315_185613/prompts/batch-5.md +0 -77
- package/.desloppify/subagents/runs/20260315_185613/prompts/batch-6.md +0 -78
- package/.desloppify/subagents/runs/20260315_185613/prompts/batch-7.md +0 -94
- package/.desloppify/subagents/runs/20260315_185613/prompts/batch-8.md +0 -84
- package/.desloppify/subagents/runs/20260315_185613/prompts/batch-9.md +0 -112
- package/.desloppify/subagents/runs/20260315_185613/results/batch-1.raw.txt +0 -78
- package/.desloppify/subagents/runs/20260315_185613/results/batch-10.raw.txt +0 -242
- package/.desloppify/subagents/runs/20260315_185613/results/batch-2.raw.txt +0 -102
- package/.desloppify/subagents/runs/20260315_185613/results/batch-3.raw.txt +0 -94
- package/.desloppify/subagents/runs/20260315_185613/results/batch-4.raw.txt +0 -86
- package/.desloppify/subagents/runs/20260315_185613/results/batch-5.raw.txt +0 -1
- package/.desloppify/subagents/runs/20260315_185613/results/batch-6.raw.txt +0 -87
- package/.desloppify/subagents/runs/20260315_185613/results/batch-7.raw.txt +0 -1
- package/.desloppify/subagents/runs/20260315_185613/results/batch-8.raw.txt +0 -107
- package/.desloppify/subagents/runs/20260315_185613/results/batch-9.raw.txt +0 -67
- package/.desloppify/subagents/runs/20260315_185613/run.log +0 -96
- package/.desloppify/subagents/runs/20260315_185613/run_summary.json +0 -156
- package/.editorconfig +0 -12
- package/.github/dependabot.yml +0 -11
- package/.github/workflows/docs.yml +0 -46
- package/.github/workflows/test.yml +0 -40
- package/.markdownlint.jsonc +0 -11
- package/.node-version +0 -1
- package/.oxfmtrc.json +0 -9
- package/.oxlintrc.json +0 -35
- package/docs/ARCHITECTURE.md +0 -125
- package/scorecard.png +0 -0
- package/src/cli/cli.ts +0 -141
- package/src/cli/validators.ts +0 -68
- package/src/formats/anthropic/index.ts +0 -14
- package/src/formats/anthropic/parse.ts +0 -70
- package/src/formats/anthropic/schema.ts +0 -74
- package/src/formats/anthropic/serialize.ts +0 -179
- package/src/formats/openai/chat-completions/index.ts +0 -14
- package/src/formats/openai/chat-completions/parse.ts +0 -47
- package/src/formats/openai/chat-completions/schema.ts +0 -92
- package/src/formats/openai/chat-completions/serialize.ts +0 -146
- package/src/formats/openai/responses/index.ts +0 -14
- package/src/formats/openai/responses/parse.ts +0 -73
- package/src/formats/openai/responses/schema.ts +0 -86
- package/src/formats/openai/responses/serialize.ts +0 -328
- package/src/formats/request-helpers.ts +0 -56
- package/src/formats/serialize-helpers.ts +0 -43
- package/src/formats/types.ts +0 -26
- package/src/history.ts +0 -70
- package/src/index.ts +0 -46
- package/src/loader.ts +0 -246
- package/src/logger.ts +0 -70
- package/src/mock-server.ts +0 -203
- package/src/route-handler.ts +0 -144
- package/src/rule-builder.ts +0 -73
- package/src/rule-engine.ts +0 -165
- package/src/sse-writer.ts +0 -35
- package/src/types/reply.ts +0 -92
- package/src/types/request.ts +0 -56
- package/src/types/rule.ts +0 -125
- package/src/types.ts +0 -24
- package/test/cli-validators.test.ts +0 -151
- package/test/formats/anthropic.test.ts +0 -336
- package/test/formats/openai.test.ts +0 -316
- package/test/formats/parse-helpers.test.ts +0 -315
- package/test/formats/responses.test.ts +0 -380
- package/test/helpers/make-req.ts +0 -18
- package/test/history.test.ts +0 -361
- package/test/loader.test.ts +0 -333
- package/test/logger.test.ts +0 -344
- package/test/mock-server.test.ts +0 -619
- package/test/rule-engine.test.ts +0 -229
- package/tsconfig.json +0 -24
- package/tsconfig.test.json +0 -11
- package/typedoc.json +0 -9
- package/vitest.config.ts +0 -18
|
@@ -1,108 +0,0 @@
|
|
|
1
|
-
You are a focused subagent reviewer for a single holistic investigation batch.
|
|
2
|
-
|
|
3
|
-
Repository root: /Users/suyash.x.srijan/Documents/Personal_Projects/llm-mock-server
|
|
4
|
-
Blind packet: /Users/suyash.x.srijan/Documents/Personal_Projects/llm-mock-server/.desloppify/review_packet_blind.json
|
|
5
|
-
Batch index: 10
|
|
6
|
-
Batch name: Full Codebase Sweep
|
|
7
|
-
Batch dimensions: cross_module_architecture, convention_outlier, error_consistency, abstraction_fitness, api_surface_coherence, authorization_consistency, ai_generated_debt, incomplete_migration, package_organization, high_level_elegance, mid_level_elegance, low_level_elegance, design_coherence
|
|
8
|
-
Batch rationale: thorough default: evaluate cross-cutting quality across all production files
|
|
9
|
-
|
|
10
|
-
Files assigned:
|
|
11
|
-
- src/cli-validators.ts
|
|
12
|
-
- src/cli.ts
|
|
13
|
-
- src/formats/anthropic/index.ts
|
|
14
|
-
- src/formats/anthropic/parse.ts
|
|
15
|
-
- src/formats/anthropic/schema.ts
|
|
16
|
-
- src/formats/anthropic/serialize.ts
|
|
17
|
-
- src/formats/openai/index.ts
|
|
18
|
-
- src/formats/openai/parse.ts
|
|
19
|
-
- src/formats/openai/schema.ts
|
|
20
|
-
- src/formats/openai/serialize.ts
|
|
21
|
-
- src/formats/request-helpers.ts
|
|
22
|
-
- src/formats/responses/index.ts
|
|
23
|
-
- src/formats/responses/parse.ts
|
|
24
|
-
- src/formats/responses/schema.ts
|
|
25
|
-
- src/formats/responses/serialize.ts
|
|
26
|
-
- src/formats/serialize-helpers.ts
|
|
27
|
-
- src/formats/types.ts
|
|
28
|
-
- src/history.ts
|
|
29
|
-
- src/index.ts
|
|
30
|
-
- src/loader.ts
|
|
31
|
-
- src/logger.ts
|
|
32
|
-
- src/mock-server.ts
|
|
33
|
-
- src/route-handler.ts
|
|
34
|
-
- src/rule-engine.ts
|
|
35
|
-
- src/sse-writer.ts
|
|
36
|
-
- src/types.ts
|
|
37
|
-
- src/types/reply.ts
|
|
38
|
-
- src/types/request.ts
|
|
39
|
-
- src/types/rule.ts
|
|
40
|
-
- vitest.config.ts
|
|
41
|
-
|
|
42
|
-
Task requirements:
|
|
43
|
-
1. Read the blind packet and follow `system_prompt` constraints exactly.
|
|
44
|
-
1a. If previously flagged issues are listed above, use them as context for your review.
|
|
45
|
-
Verify whether each still applies to the current code. Do not re-report fixed or
|
|
46
|
-
wontfix issues. Use them as starting points to look deeper — inspect adjacent code
|
|
47
|
-
and related modules for defects the prior review may have missed.
|
|
48
|
-
1c. Think structurally: when you spot multiple individual issues that share a common
|
|
49
|
-
root cause (missing abstraction, duplicated pattern, inconsistent convention),
|
|
50
|
-
explain the deeper structural issue in the finding, not just the surface symptom.
|
|
51
|
-
If the pattern is significant enough, report the structural issue as its own finding
|
|
52
|
-
with appropriate fix_scope ('multi_file_refactor' or 'architectural_change') and
|
|
53
|
-
use `root_cause_cluster` to connect related symptom findings together.
|
|
54
|
-
2. Evaluate ONLY listed files and ONLY listed dimensions for this batch.
|
|
55
|
-
3. Return 0-13 high-quality findings for this batch (empty array allowed).
|
|
56
|
-
3a. Do not suppress real defects to keep scores high; report every material issue you can support with evidence.
|
|
57
|
-
3b. Do not default to 100. Reserve 100 for genuinely exemplary evidence in this batch.
|
|
58
|
-
4. Score/finding consistency is required: broader or more severe findings MUST lower dimension scores.
|
|
59
|
-
4a. Any dimension scored below 85.0 MUST include explicit feedback: add at least one finding with the same `dimension` and a non-empty actionable `suggestion`.
|
|
60
|
-
5. Every finding must include `related_files` with at least 2 files when possible.
|
|
61
|
-
6. Every finding must include `dimension`, `identifier`, `summary`, `evidence`, `suggestion`, and `confidence`.
|
|
62
|
-
7. Every finding must include `impact_scope` and `fix_scope`.
|
|
63
|
-
8. Every scored dimension MUST include dimension_notes with concrete evidence.
|
|
64
|
-
9. If a dimension score is >85.0, include `issues_preventing_higher_score` in dimension_notes.
|
|
65
|
-
10. Use exactly one decimal place for every assessment and abstraction sub-axis score.
|
|
66
|
-
9a. For package_organization, ground scoring in objective structure signals from `holistic_context.structure` (root_files fan_in/fan_out roles, directory_profiles, coupling_matrix). Prefer thresholded evidence (for example: fan_in < 5 for root stragglers, import-affinity > 60%, directories > 10 files with mixed concerns).
|
|
67
|
-
9b. Suggestions must include a staged reorg plan (target folders, move order, and import-update/validation commands).
|
|
68
|
-
11. Ignore prior chat context and any target-threshold assumptions.
|
|
69
|
-
12. Do not edit repository files.
|
|
70
|
-
13. Return ONLY valid JSON, no markdown fences.
|
|
71
|
-
|
|
72
|
-
Scope enums:
|
|
73
|
-
- impact_scope: "local" | "module" | "subsystem" | "codebase"
|
|
74
|
-
- fix_scope: "single_edit" | "multi_file_refactor" | "architectural_change"
|
|
75
|
-
|
|
76
|
-
Output schema:
|
|
77
|
-
{
|
|
78
|
-
"batch": "Full Codebase Sweep",
|
|
79
|
-
"batch_index": 10,
|
|
80
|
-
"assessments": {"<dimension>": <0-100 with one decimal place>},
|
|
81
|
-
"dimension_notes": {
|
|
82
|
-
"<dimension>": {
|
|
83
|
-
"evidence": ["specific code observations"],
|
|
84
|
-
"impact_scope": "local|module|subsystem|codebase",
|
|
85
|
-
"fix_scope": "single_edit|multi_file_refactor|architectural_change",
|
|
86
|
-
"confidence": "high|medium|low",
|
|
87
|
-
"issues_preventing_higher_score": "required when score >85.0",
|
|
88
|
-
"sub_axes": {"abstraction_leverage": 0-100 with one decimal place, "indirection_cost": 0-100 with one decimal place, "interface_honesty": 0-100 with one decimal place} // required for abstraction_fitness when evidence supports it
|
|
89
|
-
}
|
|
90
|
-
},
|
|
91
|
-
"findings": [{
|
|
92
|
-
"dimension": "<dimension>",
|
|
93
|
-
"identifier": "short_id",
|
|
94
|
-
"summary": "one-line defect summary",
|
|
95
|
-
"related_files": ["relative/path.py"],
|
|
96
|
-
"evidence": ["specific code observation"],
|
|
97
|
-
"suggestion": "concrete fix recommendation",
|
|
98
|
-
"confidence": "high|medium|low",
|
|
99
|
-
"impact_scope": "local|module|subsystem|codebase",
|
|
100
|
-
"fix_scope": "single_edit|multi_file_refactor|architectural_change",
|
|
101
|
-
"root_cause_cluster": "optional_cluster_name_when_supported_by_history"
|
|
102
|
-
}],
|
|
103
|
-
"retrospective": {
|
|
104
|
-
"root_causes": ["optional: concise root-cause hypotheses"],
|
|
105
|
-
"likely_symptoms": ["optional: identifiers that look symptom-level"],
|
|
106
|
-
"possible_false_positives": ["optional: prior concept keys likely mis-scoped"]
|
|
107
|
-
}
|
|
108
|
-
}
|
|
@@ -1,89 +0,0 @@
|
|
|
1
|
-
You are a focused subagent reviewer for a single holistic investigation batch.
|
|
2
|
-
|
|
3
|
-
Repository root: /Users/suyash.x.srijan/Documents/Personal_Projects/llm-mock-server
|
|
4
|
-
Blind packet: /Users/suyash.x.srijan/Documents/Personal_Projects/llm-mock-server/.desloppify/review_packet_blind.json
|
|
5
|
-
Batch index: 2
|
|
6
|
-
Batch name: Conventions & Errors
|
|
7
|
-
Batch dimensions: convention_outlier, error_consistency, mid_level_elegance
|
|
8
|
-
Batch rationale: naming drift, behavioral outliers, mixed error strategies
|
|
9
|
-
|
|
10
|
-
Files assigned:
|
|
11
|
-
- src/cli.ts
|
|
12
|
-
- src/formats/anthropic/schema.ts
|
|
13
|
-
- src/formats/openai/schema.ts
|
|
14
|
-
- src/formats/responses/schema.ts
|
|
15
|
-
- src/logger.ts
|
|
16
|
-
- src/types.ts
|
|
17
|
-
- src/types/request.ts
|
|
18
|
-
- test/helpers/make-req.ts
|
|
19
|
-
- test/cli-validators.test.ts
|
|
20
|
-
- test/mock-server.test.ts
|
|
21
|
-
- test/rule-engine.test.ts
|
|
22
|
-
- src/cli-validators.ts
|
|
23
|
-
- src/history.ts
|
|
24
|
-
|
|
25
|
-
Task requirements:
|
|
26
|
-
1. Read the blind packet and follow `system_prompt` constraints exactly.
|
|
27
|
-
1a. If previously flagged issues are listed above, use them as context for your review.
|
|
28
|
-
Verify whether each still applies to the current code. Do not re-report fixed or
|
|
29
|
-
wontfix issues. Use them as starting points to look deeper — inspect adjacent code
|
|
30
|
-
and related modules for defects the prior review may have missed.
|
|
31
|
-
1c. Think structurally: when you spot multiple individual issues that share a common
|
|
32
|
-
root cause (missing abstraction, duplicated pattern, inconsistent convention),
|
|
33
|
-
explain the deeper structural issue in the finding, not just the surface symptom.
|
|
34
|
-
If the pattern is significant enough, report the structural issue as its own finding
|
|
35
|
-
with appropriate fix_scope ('multi_file_refactor' or 'architectural_change') and
|
|
36
|
-
use `root_cause_cluster` to connect related symptom findings together.
|
|
37
|
-
2. Evaluate ONLY listed files and ONLY listed dimensions for this batch.
|
|
38
|
-
3. Return 0-10 high-quality findings for this batch (empty array allowed).
|
|
39
|
-
3a. Do not suppress real defects to keep scores high; report every material issue you can support with evidence.
|
|
40
|
-
3b. Do not default to 100. Reserve 100 for genuinely exemplary evidence in this batch.
|
|
41
|
-
4. Score/finding consistency is required: broader or more severe findings MUST lower dimension scores.
|
|
42
|
-
4a. Any dimension scored below 85.0 MUST include explicit feedback: add at least one finding with the same `dimension` and a non-empty actionable `suggestion`.
|
|
43
|
-
5. Every finding must include `related_files` with at least 2 files when possible.
|
|
44
|
-
6. Every finding must include `dimension`, `identifier`, `summary`, `evidence`, `suggestion`, and `confidence`.
|
|
45
|
-
7. Every finding must include `impact_scope` and `fix_scope`.
|
|
46
|
-
8. Every scored dimension MUST include dimension_notes with concrete evidence.
|
|
47
|
-
9. If a dimension score is >85.0, include `issues_preventing_higher_score` in dimension_notes.
|
|
48
|
-
10. Use exactly one decimal place for every assessment and abstraction sub-axis score.
|
|
49
|
-
11. Ignore prior chat context and any target-threshold assumptions.
|
|
50
|
-
12. Do not edit repository files.
|
|
51
|
-
13. Return ONLY valid JSON, no markdown fences.
|
|
52
|
-
|
|
53
|
-
Scope enums:
|
|
54
|
-
- impact_scope: "local" | "module" | "subsystem" | "codebase"
|
|
55
|
-
- fix_scope: "single_edit" | "multi_file_refactor" | "architectural_change"
|
|
56
|
-
|
|
57
|
-
Output schema:
|
|
58
|
-
{
|
|
59
|
-
"batch": "Conventions & Errors",
|
|
60
|
-
"batch_index": 2,
|
|
61
|
-
"assessments": {"<dimension>": <0-100 with one decimal place>},
|
|
62
|
-
"dimension_notes": {
|
|
63
|
-
"<dimension>": {
|
|
64
|
-
"evidence": ["specific code observations"],
|
|
65
|
-
"impact_scope": "local|module|subsystem|codebase",
|
|
66
|
-
"fix_scope": "single_edit|multi_file_refactor|architectural_change",
|
|
67
|
-
"confidence": "high|medium|low",
|
|
68
|
-
"issues_preventing_higher_score": "required when score >85.0",
|
|
69
|
-
"sub_axes": {"abstraction_leverage": 0-100 with one decimal place, "indirection_cost": 0-100 with one decimal place, "interface_honesty": 0-100 with one decimal place} // required for abstraction_fitness when evidence supports it
|
|
70
|
-
}
|
|
71
|
-
},
|
|
72
|
-
"findings": [{
|
|
73
|
-
"dimension": "<dimension>",
|
|
74
|
-
"identifier": "short_id",
|
|
75
|
-
"summary": "one-line defect summary",
|
|
76
|
-
"related_files": ["relative/path.py"],
|
|
77
|
-
"evidence": ["specific code observation"],
|
|
78
|
-
"suggestion": "concrete fix recommendation",
|
|
79
|
-
"confidence": "high|medium|low",
|
|
80
|
-
"impact_scope": "local|module|subsystem|codebase",
|
|
81
|
-
"fix_scope": "single_edit|multi_file_refactor|architectural_change",
|
|
82
|
-
"root_cause_cluster": "optional_cluster_name_when_supported_by_history"
|
|
83
|
-
}],
|
|
84
|
-
"retrospective": {
|
|
85
|
-
"root_causes": ["optional: concise root-cause hypotheses"],
|
|
86
|
-
"likely_symptoms": ["optional: identifiers that look symptom-level"],
|
|
87
|
-
"possible_false_positives": ["optional: prior concept keys likely mis-scoped"]
|
|
88
|
-
}
|
|
89
|
-
}
|
|
@@ -1,91 +0,0 @@
|
|
|
1
|
-
You are a focused subagent reviewer for a single holistic investigation batch.
|
|
2
|
-
|
|
3
|
-
Repository root: /Users/suyash.x.srijan/Documents/Personal_Projects/llm-mock-server
|
|
4
|
-
Blind packet: /Users/suyash.x.srijan/Documents/Personal_Projects/llm-mock-server/.desloppify/review_packet_blind.json
|
|
5
|
-
Batch index: 3
|
|
6
|
-
Batch name: Abstractions & Dependencies
|
|
7
|
-
Batch dimensions: abstraction_fitness, mid_level_elegance, low_level_elegance
|
|
8
|
-
Batch rationale: abstraction hotspots (wrappers/interfaces/param bags), dep cycles
|
|
9
|
-
|
|
10
|
-
Files assigned:
|
|
11
|
-
- test/cli-validators.test.ts
|
|
12
|
-
- src/mock-server.ts
|
|
13
|
-
- test/formats/anthropic.test.ts
|
|
14
|
-
- src/cli.ts
|
|
15
|
-
- test/mock-server.test.ts
|
|
16
|
-
- src/formats/responses/serialize.ts
|
|
17
|
-
- test/formats/responses.test.ts
|
|
18
|
-
- test/history.test.ts
|
|
19
|
-
- src/rule-engine.ts
|
|
20
|
-
- src/history.ts
|
|
21
|
-
- src/loader.ts
|
|
22
|
-
- test/formats/openai.test.ts
|
|
23
|
-
- test/loader.test.ts
|
|
24
|
-
- test/logger.test.ts
|
|
25
|
-
- src/formats/openai/serialize.ts
|
|
26
|
-
|
|
27
|
-
Task requirements:
|
|
28
|
-
1. Read the blind packet and follow `system_prompt` constraints exactly.
|
|
29
|
-
1a. If previously flagged issues are listed above, use them as context for your review.
|
|
30
|
-
Verify whether each still applies to the current code. Do not re-report fixed or
|
|
31
|
-
wontfix issues. Use them as starting points to look deeper — inspect adjacent code
|
|
32
|
-
and related modules for defects the prior review may have missed.
|
|
33
|
-
1c. Think structurally: when you spot multiple individual issues that share a common
|
|
34
|
-
root cause (missing abstraction, duplicated pattern, inconsistent convention),
|
|
35
|
-
explain the deeper structural issue in the finding, not just the surface symptom.
|
|
36
|
-
If the pattern is significant enough, report the structural issue as its own finding
|
|
37
|
-
with appropriate fix_scope ('multi_file_refactor' or 'architectural_change') and
|
|
38
|
-
use `root_cause_cluster` to connect related symptom findings together.
|
|
39
|
-
2. Evaluate ONLY listed files and ONLY listed dimensions for this batch.
|
|
40
|
-
3. Return 0-10 high-quality findings for this batch (empty array allowed).
|
|
41
|
-
3a. Do not suppress real defects to keep scores high; report every material issue you can support with evidence.
|
|
42
|
-
3b. Do not default to 100. Reserve 100 for genuinely exemplary evidence in this batch.
|
|
43
|
-
4. Score/finding consistency is required: broader or more severe findings MUST lower dimension scores.
|
|
44
|
-
4a. Any dimension scored below 85.0 MUST include explicit feedback: add at least one finding with the same `dimension` and a non-empty actionable `suggestion`.
|
|
45
|
-
5. Every finding must include `related_files` with at least 2 files when possible.
|
|
46
|
-
6. Every finding must include `dimension`, `identifier`, `summary`, `evidence`, `suggestion`, and `confidence`.
|
|
47
|
-
7. Every finding must include `impact_scope` and `fix_scope`.
|
|
48
|
-
8. Every scored dimension MUST include dimension_notes with concrete evidence.
|
|
49
|
-
9. If a dimension score is >85.0, include `issues_preventing_higher_score` in dimension_notes.
|
|
50
|
-
10. Use exactly one decimal place for every assessment and abstraction sub-axis score.
|
|
51
|
-
11. Ignore prior chat context and any target-threshold assumptions.
|
|
52
|
-
12. Do not edit repository files.
|
|
53
|
-
13. Return ONLY valid JSON, no markdown fences.
|
|
54
|
-
|
|
55
|
-
Scope enums:
|
|
56
|
-
- impact_scope: "local" | "module" | "subsystem" | "codebase"
|
|
57
|
-
- fix_scope: "single_edit" | "multi_file_refactor" | "architectural_change"
|
|
58
|
-
|
|
59
|
-
Output schema:
|
|
60
|
-
{
|
|
61
|
-
"batch": "Abstractions & Dependencies",
|
|
62
|
-
"batch_index": 3,
|
|
63
|
-
"assessments": {"<dimension>": <0-100 with one decimal place>},
|
|
64
|
-
"dimension_notes": {
|
|
65
|
-
"<dimension>": {
|
|
66
|
-
"evidence": ["specific code observations"],
|
|
67
|
-
"impact_scope": "local|module|subsystem|codebase",
|
|
68
|
-
"fix_scope": "single_edit|multi_file_refactor|architectural_change",
|
|
69
|
-
"confidence": "high|medium|low",
|
|
70
|
-
"issues_preventing_higher_score": "required when score >85.0",
|
|
71
|
-
"sub_axes": {"abstraction_leverage": 0-100 with one decimal place, "indirection_cost": 0-100 with one decimal place, "interface_honesty": 0-100 with one decimal place} // required for abstraction_fitness when evidence supports it
|
|
72
|
-
}
|
|
73
|
-
},
|
|
74
|
-
"findings": [{
|
|
75
|
-
"dimension": "<dimension>",
|
|
76
|
-
"identifier": "short_id",
|
|
77
|
-
"summary": "one-line defect summary",
|
|
78
|
-
"related_files": ["relative/path.py"],
|
|
79
|
-
"evidence": ["specific code observation"],
|
|
80
|
-
"suggestion": "concrete fix recommendation",
|
|
81
|
-
"confidence": "high|medium|low",
|
|
82
|
-
"impact_scope": "local|module|subsystem|codebase",
|
|
83
|
-
"fix_scope": "single_edit|multi_file_refactor|architectural_change",
|
|
84
|
-
"root_cause_cluster": "optional_cluster_name_when_supported_by_history"
|
|
85
|
-
}],
|
|
86
|
-
"retrospective": {
|
|
87
|
-
"root_causes": ["optional: concise root-cause hypotheses"],
|
|
88
|
-
"likely_symptoms": ["optional: identifiers that look symptom-level"],
|
|
89
|
-
"possible_false_positives": ["optional: prior concept keys likely mis-scoped"]
|
|
90
|
-
}
|
|
91
|
-
}
|
|
@@ -1,77 +0,0 @@
|
|
|
1
|
-
You are a focused subagent reviewer for a single holistic investigation batch.
|
|
2
|
-
|
|
3
|
-
Repository root: /Users/suyash.x.srijan/Documents/Personal_Projects/llm-mock-server
|
|
4
|
-
Blind packet: /Users/suyash.x.srijan/Documents/Personal_Projects/llm-mock-server/.desloppify/review_packet_blind.json
|
|
5
|
-
Batch index: 4
|
|
6
|
-
Batch name: Testing & API
|
|
7
|
-
Batch dimensions: api_surface_coherence, mid_level_elegance
|
|
8
|
-
Batch rationale: critical untested paths, API inconsistency
|
|
9
|
-
|
|
10
|
-
Files assigned:
|
|
11
|
-
- src/cli-validators.ts
|
|
12
|
-
|
|
13
|
-
Task requirements:
|
|
14
|
-
1. Read the blind packet and follow `system_prompt` constraints exactly.
|
|
15
|
-
1a. If previously flagged issues are listed above, use them as context for your review.
|
|
16
|
-
Verify whether each still applies to the current code. Do not re-report fixed or
|
|
17
|
-
wontfix issues. Use them as starting points to look deeper — inspect adjacent code
|
|
18
|
-
and related modules for defects the prior review may have missed.
|
|
19
|
-
1c. Think structurally: when you spot multiple individual issues that share a common
|
|
20
|
-
root cause (missing abstraction, duplicated pattern, inconsistent convention),
|
|
21
|
-
explain the deeper structural issue in the finding, not just the surface symptom.
|
|
22
|
-
If the pattern is significant enough, report the structural issue as its own finding
|
|
23
|
-
with appropriate fix_scope ('multi_file_refactor' or 'architectural_change') and
|
|
24
|
-
use `root_cause_cluster` to connect related symptom findings together.
|
|
25
|
-
2. Evaluate ONLY listed files and ONLY listed dimensions for this batch.
|
|
26
|
-
3. Return 0-10 high-quality findings for this batch (empty array allowed).
|
|
27
|
-
3a. Do not suppress real defects to keep scores high; report every material issue you can support with evidence.
|
|
28
|
-
3b. Do not default to 100. Reserve 100 for genuinely exemplary evidence in this batch.
|
|
29
|
-
4. Score/finding consistency is required: broader or more severe findings MUST lower dimension scores.
|
|
30
|
-
4a. Any dimension scored below 85.0 MUST include explicit feedback: add at least one finding with the same `dimension` and a non-empty actionable `suggestion`.
|
|
31
|
-
5. Every finding must include `related_files` with at least 2 files when possible.
|
|
32
|
-
6. Every finding must include `dimension`, `identifier`, `summary`, `evidence`, `suggestion`, and `confidence`.
|
|
33
|
-
7. Every finding must include `impact_scope` and `fix_scope`.
|
|
34
|
-
8. Every scored dimension MUST include dimension_notes with concrete evidence.
|
|
35
|
-
9. If a dimension score is >85.0, include `issues_preventing_higher_score` in dimension_notes.
|
|
36
|
-
10. Use exactly one decimal place for every assessment and abstraction sub-axis score.
|
|
37
|
-
11. Ignore prior chat context and any target-threshold assumptions.
|
|
38
|
-
12. Do not edit repository files.
|
|
39
|
-
13. Return ONLY valid JSON, no markdown fences.
|
|
40
|
-
|
|
41
|
-
Scope enums:
|
|
42
|
-
- impact_scope: "local" | "module" | "subsystem" | "codebase"
|
|
43
|
-
- fix_scope: "single_edit" | "multi_file_refactor" | "architectural_change"
|
|
44
|
-
|
|
45
|
-
Output schema:
|
|
46
|
-
{
|
|
47
|
-
"batch": "Testing & API",
|
|
48
|
-
"batch_index": 4,
|
|
49
|
-
"assessments": {"<dimension>": <0-100 with one decimal place>},
|
|
50
|
-
"dimension_notes": {
|
|
51
|
-
"<dimension>": {
|
|
52
|
-
"evidence": ["specific code observations"],
|
|
53
|
-
"impact_scope": "local|module|subsystem|codebase",
|
|
54
|
-
"fix_scope": "single_edit|multi_file_refactor|architectural_change",
|
|
55
|
-
"confidence": "high|medium|low",
|
|
56
|
-
"issues_preventing_higher_score": "required when score >85.0",
|
|
57
|
-
"sub_axes": {"abstraction_leverage": 0-100 with one decimal place, "indirection_cost": 0-100 with one decimal place, "interface_honesty": 0-100 with one decimal place} // required for abstraction_fitness when evidence supports it
|
|
58
|
-
}
|
|
59
|
-
},
|
|
60
|
-
"findings": [{
|
|
61
|
-
"dimension": "<dimension>",
|
|
62
|
-
"identifier": "short_id",
|
|
63
|
-
"summary": "one-line defect summary",
|
|
64
|
-
"related_files": ["relative/path.py"],
|
|
65
|
-
"evidence": ["specific code observation"],
|
|
66
|
-
"suggestion": "concrete fix recommendation",
|
|
67
|
-
"confidence": "high|medium|low",
|
|
68
|
-
"impact_scope": "local|module|subsystem|codebase",
|
|
69
|
-
"fix_scope": "single_edit|multi_file_refactor|architectural_change",
|
|
70
|
-
"root_cause_cluster": "optional_cluster_name_when_supported_by_history"
|
|
71
|
-
}],
|
|
72
|
-
"retrospective": {
|
|
73
|
-
"root_causes": ["optional: concise root-cause hypotheses"],
|
|
74
|
-
"likely_symptoms": ["optional: identifiers that look symptom-level"],
|
|
75
|
-
"possible_false_positives": ["optional: prior concept keys likely mis-scoped"]
|
|
76
|
-
}
|
|
77
|
-
}
|
|
@@ -1,77 +0,0 @@
|
|
|
1
|
-
You are a focused subagent reviewer for a single holistic investigation batch.
|
|
2
|
-
|
|
3
|
-
Repository root: /Users/suyash.x.srijan/Documents/Personal_Projects/llm-mock-server
|
|
4
|
-
Blind packet: /Users/suyash.x.srijan/Documents/Personal_Projects/llm-mock-server/.desloppify/review_packet_blind.json
|
|
5
|
-
Batch index: 5
|
|
6
|
-
Batch name: Authorization
|
|
7
|
-
Batch dimensions: authorization_consistency, mid_level_elegance
|
|
8
|
-
Batch rationale: auth gaps, service role usage, RLS coverage
|
|
9
|
-
|
|
10
|
-
Files assigned:
|
|
11
|
-
- src/mock-server.ts
|
|
12
|
-
|
|
13
|
-
Task requirements:
|
|
14
|
-
1. Read the blind packet and follow `system_prompt` constraints exactly.
|
|
15
|
-
1a. If previously flagged issues are listed above, use them as context for your review.
|
|
16
|
-
Verify whether each still applies to the current code. Do not re-report fixed or
|
|
17
|
-
wontfix issues. Use them as starting points to look deeper — inspect adjacent code
|
|
18
|
-
and related modules for defects the prior review may have missed.
|
|
19
|
-
1c. Think structurally: when you spot multiple individual issues that share a common
|
|
20
|
-
root cause (missing abstraction, duplicated pattern, inconsistent convention),
|
|
21
|
-
explain the deeper structural issue in the finding, not just the surface symptom.
|
|
22
|
-
If the pattern is significant enough, report the structural issue as its own finding
|
|
23
|
-
with appropriate fix_scope ('multi_file_refactor' or 'architectural_change') and
|
|
24
|
-
use `root_cause_cluster` to connect related symptom findings together.
|
|
25
|
-
2. Evaluate ONLY listed files and ONLY listed dimensions for this batch.
|
|
26
|
-
3. Return 0-10 high-quality findings for this batch (empty array allowed).
|
|
27
|
-
3a. Do not suppress real defects to keep scores high; report every material issue you can support with evidence.
|
|
28
|
-
3b. Do not default to 100. Reserve 100 for genuinely exemplary evidence in this batch.
|
|
29
|
-
4. Score/finding consistency is required: broader or more severe findings MUST lower dimension scores.
|
|
30
|
-
4a. Any dimension scored below 85.0 MUST include explicit feedback: add at least one finding with the same `dimension` and a non-empty actionable `suggestion`.
|
|
31
|
-
5. Every finding must include `related_files` with at least 2 files when possible.
|
|
32
|
-
6. Every finding must include `dimension`, `identifier`, `summary`, `evidence`, `suggestion`, and `confidence`.
|
|
33
|
-
7. Every finding must include `impact_scope` and `fix_scope`.
|
|
34
|
-
8. Every scored dimension MUST include dimension_notes with concrete evidence.
|
|
35
|
-
9. If a dimension score is >85.0, include `issues_preventing_higher_score` in dimension_notes.
|
|
36
|
-
10. Use exactly one decimal place for every assessment and abstraction sub-axis score.
|
|
37
|
-
11. Ignore prior chat context and any target-threshold assumptions.
|
|
38
|
-
12. Do not edit repository files.
|
|
39
|
-
13. Return ONLY valid JSON, no markdown fences.
|
|
40
|
-
|
|
41
|
-
Scope enums:
|
|
42
|
-
- impact_scope: "local" | "module" | "subsystem" | "codebase"
|
|
43
|
-
- fix_scope: "single_edit" | "multi_file_refactor" | "architectural_change"
|
|
44
|
-
|
|
45
|
-
Output schema:
|
|
46
|
-
{
|
|
47
|
-
"batch": "Authorization",
|
|
48
|
-
"batch_index": 5,
|
|
49
|
-
"assessments": {"<dimension>": <0-100 with one decimal place>},
|
|
50
|
-
"dimension_notes": {
|
|
51
|
-
"<dimension>": {
|
|
52
|
-
"evidence": ["specific code observations"],
|
|
53
|
-
"impact_scope": "local|module|subsystem|codebase",
|
|
54
|
-
"fix_scope": "single_edit|multi_file_refactor|architectural_change",
|
|
55
|
-
"confidence": "high|medium|low",
|
|
56
|
-
"issues_preventing_higher_score": "required when score >85.0",
|
|
57
|
-
"sub_axes": {"abstraction_leverage": 0-100 with one decimal place, "indirection_cost": 0-100 with one decimal place, "interface_honesty": 0-100 with one decimal place} // required for abstraction_fitness when evidence supports it
|
|
58
|
-
}
|
|
59
|
-
},
|
|
60
|
-
"findings": [{
|
|
61
|
-
"dimension": "<dimension>",
|
|
62
|
-
"identifier": "short_id",
|
|
63
|
-
"summary": "one-line defect summary",
|
|
64
|
-
"related_files": ["relative/path.py"],
|
|
65
|
-
"evidence": ["specific code observation"],
|
|
66
|
-
"suggestion": "concrete fix recommendation",
|
|
67
|
-
"confidence": "high|medium|low",
|
|
68
|
-
"impact_scope": "local|module|subsystem|codebase",
|
|
69
|
-
"fix_scope": "single_edit|multi_file_refactor|architectural_change",
|
|
70
|
-
"root_cause_cluster": "optional_cluster_name_when_supported_by_history"
|
|
71
|
-
}],
|
|
72
|
-
"retrospective": {
|
|
73
|
-
"root_causes": ["optional: concise root-cause hypotheses"],
|
|
74
|
-
"likely_symptoms": ["optional: identifiers that look symptom-level"],
|
|
75
|
-
"possible_false_positives": ["optional: prior concept keys likely mis-scoped"]
|
|
76
|
-
}
|
|
77
|
-
}
|
|
@@ -1,78 +0,0 @@
|
|
|
1
|
-
You are a focused subagent reviewer for a single holistic investigation batch.
|
|
2
|
-
|
|
3
|
-
Repository root: /Users/suyash.x.srijan/Documents/Personal_Projects/llm-mock-server
|
|
4
|
-
Blind packet: /Users/suyash.x.srijan/Documents/Personal_Projects/llm-mock-server/.desloppify/review_packet_blind.json
|
|
5
|
-
Batch index: 6
|
|
6
|
-
Batch name: AI Debt & Migrations
|
|
7
|
-
Batch dimensions: ai_generated_debt, incomplete_migration, low_level_elegance
|
|
8
|
-
Batch rationale: AI-generated patterns, deprecated markers, migration TODOs
|
|
9
|
-
|
|
10
|
-
Files assigned:
|
|
11
|
-
- src/types/request.ts
|
|
12
|
-
- src/types/rule.ts
|
|
13
|
-
|
|
14
|
-
Task requirements:
|
|
15
|
-
1. Read the blind packet and follow `system_prompt` constraints exactly.
|
|
16
|
-
1a. If previously flagged issues are listed above, use them as context for your review.
|
|
17
|
-
Verify whether each still applies to the current code. Do not re-report fixed or
|
|
18
|
-
wontfix issues. Use them as starting points to look deeper — inspect adjacent code
|
|
19
|
-
and related modules for defects the prior review may have missed.
|
|
20
|
-
1c. Think structurally: when you spot multiple individual issues that share a common
|
|
21
|
-
root cause (missing abstraction, duplicated pattern, inconsistent convention),
|
|
22
|
-
explain the deeper structural issue in the finding, not just the surface symptom.
|
|
23
|
-
If the pattern is significant enough, report the structural issue as its own finding
|
|
24
|
-
with appropriate fix_scope ('multi_file_refactor' or 'architectural_change') and
|
|
25
|
-
use `root_cause_cluster` to connect related symptom findings together.
|
|
26
|
-
2. Evaluate ONLY listed files and ONLY listed dimensions for this batch.
|
|
27
|
-
3. Return 0-10 high-quality findings for this batch (empty array allowed).
|
|
28
|
-
3a. Do not suppress real defects to keep scores high; report every material issue you can support with evidence.
|
|
29
|
-
3b. Do not default to 100. Reserve 100 for genuinely exemplary evidence in this batch.
|
|
30
|
-
4. Score/finding consistency is required: broader or more severe findings MUST lower dimension scores.
|
|
31
|
-
4a. Any dimension scored below 85.0 MUST include explicit feedback: add at least one finding with the same `dimension` and a non-empty actionable `suggestion`.
|
|
32
|
-
5. Every finding must include `related_files` with at least 2 files when possible.
|
|
33
|
-
6. Every finding must include `dimension`, `identifier`, `summary`, `evidence`, `suggestion`, and `confidence`.
|
|
34
|
-
7. Every finding must include `impact_scope` and `fix_scope`.
|
|
35
|
-
8. Every scored dimension MUST include dimension_notes with concrete evidence.
|
|
36
|
-
9. If a dimension score is >85.0, include `issues_preventing_higher_score` in dimension_notes.
|
|
37
|
-
10. Use exactly one decimal place for every assessment and abstraction sub-axis score.
|
|
38
|
-
11. Ignore prior chat context and any target-threshold assumptions.
|
|
39
|
-
12. Do not edit repository files.
|
|
40
|
-
13. Return ONLY valid JSON, no markdown fences.
|
|
41
|
-
|
|
42
|
-
Scope enums:
|
|
43
|
-
- impact_scope: "local" | "module" | "subsystem" | "codebase"
|
|
44
|
-
- fix_scope: "single_edit" | "multi_file_refactor" | "architectural_change"
|
|
45
|
-
|
|
46
|
-
Output schema:
|
|
47
|
-
{
|
|
48
|
-
"batch": "AI Debt & Migrations",
|
|
49
|
-
"batch_index": 6,
|
|
50
|
-
"assessments": {"<dimension>": <0-100 with one decimal place>},
|
|
51
|
-
"dimension_notes": {
|
|
52
|
-
"<dimension>": {
|
|
53
|
-
"evidence": ["specific code observations"],
|
|
54
|
-
"impact_scope": "local|module|subsystem|codebase",
|
|
55
|
-
"fix_scope": "single_edit|multi_file_refactor|architectural_change",
|
|
56
|
-
"confidence": "high|medium|low",
|
|
57
|
-
"issues_preventing_higher_score": "required when score >85.0",
|
|
58
|
-
"sub_axes": {"abstraction_leverage": 0-100 with one decimal place, "indirection_cost": 0-100 with one decimal place, "interface_honesty": 0-100 with one decimal place} // required for abstraction_fitness when evidence supports it
|
|
59
|
-
}
|
|
60
|
-
},
|
|
61
|
-
"findings": [{
|
|
62
|
-
"dimension": "<dimension>",
|
|
63
|
-
"identifier": "short_id",
|
|
64
|
-
"summary": "one-line defect summary",
|
|
65
|
-
"related_files": ["relative/path.py"],
|
|
66
|
-
"evidence": ["specific code observation"],
|
|
67
|
-
"suggestion": "concrete fix recommendation",
|
|
68
|
-
"confidence": "high|medium|low",
|
|
69
|
-
"impact_scope": "local|module|subsystem|codebase",
|
|
70
|
-
"fix_scope": "single_edit|multi_file_refactor|architectural_change",
|
|
71
|
-
"root_cause_cluster": "optional_cluster_name_when_supported_by_history"
|
|
72
|
-
}],
|
|
73
|
-
"retrospective": {
|
|
74
|
-
"root_causes": ["optional: concise root-cause hypotheses"],
|
|
75
|
-
"likely_symptoms": ["optional: identifiers that look symptom-level"],
|
|
76
|
-
"possible_false_positives": ["optional: prior concept keys likely mis-scoped"]
|
|
77
|
-
}
|
|
78
|
-
}
|
|
@@ -1,94 +0,0 @@
|
|
|
1
|
-
You are a focused subagent reviewer for a single holistic investigation batch.
|
|
2
|
-
|
|
3
|
-
Repository root: /Users/suyash.x.srijan/Documents/Personal_Projects/llm-mock-server
|
|
4
|
-
Blind packet: /Users/suyash.x.srijan/Documents/Personal_Projects/llm-mock-server/.desloppify/review_packet_blind.json
|
|
5
|
-
Batch index: 7
|
|
6
|
-
Batch name: Package Organization
|
|
7
|
-
Batch dimensions: package_organization, high_level_elegance
|
|
8
|
-
Batch rationale: file placement, directory boundaries, architectural layering
|
|
9
|
-
|
|
10
|
-
Files assigned:
|
|
11
|
-
- vitest.config.ts
|
|
12
|
-
- src/cli-validators.ts
|
|
13
|
-
- src/cli.ts
|
|
14
|
-
- src/history.ts
|
|
15
|
-
- test/cli-validators.test.ts
|
|
16
|
-
- test/history.test.ts
|
|
17
|
-
- test/loader.test.ts
|
|
18
|
-
- src/formats/anthropic/index.ts
|
|
19
|
-
- src/formats/anthropic/parse.ts
|
|
20
|
-
- src/formats/anthropic/schema.ts
|
|
21
|
-
- src/formats/request-helpers.ts
|
|
22
|
-
- src/formats/serialize-helpers.ts
|
|
23
|
-
- src/formats/openai/index.ts
|
|
24
|
-
- src/formats/openai/parse.ts
|
|
25
|
-
- src/formats/responses/index.ts
|
|
26
|
-
- src/formats/responses/parse.ts
|
|
27
|
-
|
|
28
|
-
Task requirements:
|
|
29
|
-
1. Read the blind packet and follow `system_prompt` constraints exactly.
|
|
30
|
-
1a. If previously flagged issues are listed above, use them as context for your review.
|
|
31
|
-
Verify whether each still applies to the current code. Do not re-report fixed or
|
|
32
|
-
wontfix issues. Use them as starting points to look deeper — inspect adjacent code
|
|
33
|
-
and related modules for defects the prior review may have missed.
|
|
34
|
-
1c. Think structurally: when you spot multiple individual issues that share a common
|
|
35
|
-
root cause (missing abstraction, duplicated pattern, inconsistent convention),
|
|
36
|
-
explain the deeper structural issue in the finding, not just the surface symptom.
|
|
37
|
-
If the pattern is significant enough, report the structural issue as its own finding
|
|
38
|
-
with appropriate fix_scope ('multi_file_refactor' or 'architectural_change') and
|
|
39
|
-
use `root_cause_cluster` to connect related symptom findings together.
|
|
40
|
-
2. Evaluate ONLY listed files and ONLY listed dimensions for this batch.
|
|
41
|
-
3. Return 0-10 high-quality findings for this batch (empty array allowed).
|
|
42
|
-
3a. Do not suppress real defects to keep scores high; report every material issue you can support with evidence.
|
|
43
|
-
3b. Do not default to 100. Reserve 100 for genuinely exemplary evidence in this batch.
|
|
44
|
-
4. Score/finding consistency is required: broader or more severe findings MUST lower dimension scores.
|
|
45
|
-
4a. Any dimension scored below 85.0 MUST include explicit feedback: add at least one finding with the same `dimension` and a non-empty actionable `suggestion`.
|
|
46
|
-
5. Every finding must include `related_files` with at least 2 files when possible.
|
|
47
|
-
6. Every finding must include `dimension`, `identifier`, `summary`, `evidence`, `suggestion`, and `confidence`.
|
|
48
|
-
7. Every finding must include `impact_scope` and `fix_scope`.
|
|
49
|
-
8. Every scored dimension MUST include dimension_notes with concrete evidence.
|
|
50
|
-
9. If a dimension score is >85.0, include `issues_preventing_higher_score` in dimension_notes.
|
|
51
|
-
10. Use exactly one decimal place for every assessment and abstraction sub-axis score.
|
|
52
|
-
9a. For package_organization, ground scoring in objective structure signals from `holistic_context.structure` (root_files fan_in/fan_out roles, directory_profiles, coupling_matrix). Prefer thresholded evidence (for example: fan_in < 5 for root stragglers, import-affinity > 60%, directories > 10 files with mixed concerns).
|
|
53
|
-
9b. Suggestions must include a staged reorg plan (target folders, move order, and import-update/validation commands).
|
|
54
|
-
11. Ignore prior chat context and any target-threshold assumptions.
|
|
55
|
-
12. Do not edit repository files.
|
|
56
|
-
13. Return ONLY valid JSON, no markdown fences.
|
|
57
|
-
|
|
58
|
-
Scope enums:
|
|
59
|
-
- impact_scope: "local" | "module" | "subsystem" | "codebase"
|
|
60
|
-
- fix_scope: "single_edit" | "multi_file_refactor" | "architectural_change"
|
|
61
|
-
|
|
62
|
-
Output schema:
|
|
63
|
-
{
|
|
64
|
-
"batch": "Package Organization",
|
|
65
|
-
"batch_index": 7,
|
|
66
|
-
"assessments": {"<dimension>": <0-100 with one decimal place>},
|
|
67
|
-
"dimension_notes": {
|
|
68
|
-
"<dimension>": {
|
|
69
|
-
"evidence": ["specific code observations"],
|
|
70
|
-
"impact_scope": "local|module|subsystem|codebase",
|
|
71
|
-
"fix_scope": "single_edit|multi_file_refactor|architectural_change",
|
|
72
|
-
"confidence": "high|medium|low",
|
|
73
|
-
"issues_preventing_higher_score": "required when score >85.0",
|
|
74
|
-
"sub_axes": {"abstraction_leverage": 0-100 with one decimal place, "indirection_cost": 0-100 with one decimal place, "interface_honesty": 0-100 with one decimal place} // required for abstraction_fitness when evidence supports it
|
|
75
|
-
}
|
|
76
|
-
},
|
|
77
|
-
"findings": [{
|
|
78
|
-
"dimension": "<dimension>",
|
|
79
|
-
"identifier": "short_id",
|
|
80
|
-
"summary": "one-line defect summary",
|
|
81
|
-
"related_files": ["relative/path.py"],
|
|
82
|
-
"evidence": ["specific code observation"],
|
|
83
|
-
"suggestion": "concrete fix recommendation",
|
|
84
|
-
"confidence": "high|medium|low",
|
|
85
|
-
"impact_scope": "local|module|subsystem|codebase",
|
|
86
|
-
"fix_scope": "single_edit|multi_file_refactor|architectural_change",
|
|
87
|
-
"root_cause_cluster": "optional_cluster_name_when_supported_by_history"
|
|
88
|
-
}],
|
|
89
|
-
"retrospective": {
|
|
90
|
-
"root_causes": ["optional: concise root-cause hypotheses"],
|
|
91
|
-
"likely_symptoms": ["optional: identifiers that look symptom-level"],
|
|
92
|
-
"possible_false_positives": ["optional: prior concept keys likely mis-scoped"]
|
|
93
|
-
}
|
|
94
|
-
}
|