developing-agent-forge 2.1.0 → 2.1.1

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.
@@ -1 +1 @@
1
- {"version":3,"file":"developer.d.ts","sourceRoot":"","sources":["../../src/agents/developer.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,eAAe,EAAE,KAAK,wBAAwB,EAAE,MAAM,YAAY,CAAC;AAE5E,KAAK,wBAAwB,GAAG,wBAAwB,GAAG;IACzD,SAAS,EAAE,MAAM,CAAC;IAClB,KAAK,EAAE,QAAQ,CAAC;CACjB,CAAC;AAEF,KAAK,yBAAyB,GAAG,wBAAwB,GAAG;IAC1D,SAAS,EAAE,MAAM,CAAC;IAClB,gBAAgB,EAAE,MAAM,CAAC;IACzB,KAAK,EAAE,SAAS,CAAC;IACjB,cAAc,CAAC,EAAE,MAAM,CAAC;CACzB,CAAC;AAEF,KAAK,wBAAwB,GAAG,wBAAwB,GAAG;IACzD,SAAS,EAAE,MAAM,CAAC;IAClB,gBAAgB,EAAE,MAAM,CAAC;IACzB,KAAK,EAAE,QAAQ,CAAC;IAChB,gBAAgB,EAAE,MAAM,CAAC;CAC1B,CAAC;AAEF,MAAM,MAAM,kBAAkB,GAC1B,wBAAwB,GACxB,yBAAyB,GACzB,wBAAwB,CAAC;AAE7B,qBAAa,cAAe,SAAQ,eAAe,CAAC,kBAAkB,CAAC;IACrE,SAAS,CAAC,WAAW,CAAC,SAAS,EAAE,QAAQ,CAAC,kBAAkB,CAAC,GAAG,MAAM;CAoCvE"}
1
+ {"version":3,"file":"developer.d.ts","sourceRoot":"","sources":["../../src/agents/developer.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,eAAe,EAAE,KAAK,wBAAwB,EAAE,MAAM,YAAY,CAAC;AAE5E,KAAK,wBAAwB,GAAG,wBAAwB,GAAG;IACzD,SAAS,EAAE,MAAM,CAAC;IAClB,KAAK,EAAE,QAAQ,CAAC;CACjB,CAAC;AAEF,KAAK,yBAAyB,GAAG,wBAAwB,GAAG;IAC1D,SAAS,EAAE,MAAM,CAAC;IAClB,gBAAgB,EAAE,MAAM,CAAC;IACzB,KAAK,EAAE,SAAS,CAAC;IACjB,cAAc,CAAC,EAAE,MAAM,CAAC;CACzB,CAAC;AAEF,KAAK,wBAAwB,GAAG,wBAAwB,GAAG;IACzD,SAAS,EAAE,MAAM,CAAC;IAClB,gBAAgB,EAAE,MAAM,CAAC;IACzB,KAAK,EAAE,QAAQ,CAAC;IAChB,gBAAgB,EAAE,MAAM,CAAC;CAC1B,CAAC;AAEF,MAAM,MAAM,kBAAkB,GAC1B,wBAAwB,GACxB,yBAAyB,GACzB,wBAAwB,CAAC;AAE7B,qBAAa,cAAe,SAAQ,eAAe,CAAC,kBAAkB,CAAC;IACrE,SAAS,CAAC,WAAW,CAAC,SAAS,EAAE,QAAQ,CAAC,kBAAkB,CAAC,GAAG,MAAM;CAoCvE"}
@@ -1,3 +1,4 @@
1
+ import { ponytailSkillPrompt } from "./polytail.js";
1
2
  import { codingStyleSkillInstruction, goalInstruction } from "./prompts.js";
2
3
  import { DevelopingAgent } from "./types.js";
3
4
  export class DeveloperAgent extends DevelopingAgent {
@@ -34,6 +35,8 @@ Output concise code design memory recall guidance.
34
35
  }
35
36
  function buildDevelopPrompt(codingStyleSkillInstructionText, goalInstructionText, targetPath, taskBrief, codeDesignMemory, reviewerReport) {
36
37
  return `
38
+ ${ponytailSkillPrompt}
39
+
37
40
  ${codingStyleSkillInstructionText}
38
41
 
39
42
  ${goalInstructionText}
@@ -1 +1 @@
1
- {"version":3,"file":"developer.js","sourceRoot":"","sources":["../../src/agents/developer.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,2BAA2B,EAAE,eAAe,EAAE,MAAM,cAAc,CAAC;AAC5E,OAAO,EAAE,eAAe,EAAiC,MAAM,YAAY,CAAC;AA0B5E,MAAM,OAAO,cAAe,SAAQ,eAAmC;IAC3D,WAAW,CAAC,SAAuC;QAC3D,MAAM,oBAAoB,GAAG,IAAI,CAAC,qBAAqB,CAAC,SAAS,CAAC,oBAAoB,CAAC,CAAC;QACxF,MAAM,UAAU,GAAG,IAAI,CAAC,qBAAqB,CAAC,SAAS,CAAC,UAAU,CAAC,CAAC;QACpE,MAAM,+BAA+B,GAAG,2BAA2B,CAAC,oBAAoB,CAAC,CAAC;QAC1F,MAAM,mBAAmB,GAAG,eAAe,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC;QAE5D,QAAQ,SAAS,CAAC,KAAK,EAAE,CAAC;YACxB,KAAK,QAAQ;gBACX,OAAO,iBAAiB,CACtB,+BAA+B,EAC/B,mBAAmB,EACnB,UAAU,EACV,SAAS,CAAC,SAAS,CACpB,CAAC;YACJ,KAAK,SAAS,CAAC,CAAC,CAAC;gBACf,MAAM,cAAc,GAAG,SAAS,CAAC,cAAc,IAAI,QAAQ,CAAC;gBAC5D,OAAO,kBAAkB,CACvB,+BAA+B,EAC/B,mBAAmB,EACnB,UAAU,EACV,SAAS,CAAC,SAAS,EACnB,SAAS,CAAC,gBAAgB,EAC1B,cAAc,CACf,CAAC;YACJ,CAAC;YACD,KAAK,QAAQ;gBACX,OAAO,iBAAiB,CACtB,+BAA+B,EAC/B,mBAAmB,EACnB,UAAU,EACV,SAAS,CAAC,SAAS,EACnB,SAAS,CAAC,gBAAgB,EAC1B,SAAS,CAAC,gBAAgB,CAC3B,CAAC;QACN,CAAC;IACH,CAAC;CACF;AAED,SAAS,iBAAiB,CACxB,+BAAuC,EACvC,mBAA2B,EAC3B,UAAkB,EAClB,SAAiB;IAEjB,OAAO;EACP,+BAA+B;;EAE/B,mBAAmB;;;EAGnB,SAAS;;gCAEqB,UAAU;;;CAGzC,CAAC;AACF,CAAC;AAED,SAAS,kBAAkB,CACzB,+BAAuC,EACvC,mBAA2B,EAC3B,UAAkB,EAClB,SAAiB,EACjB,gBAAwB,EACxB,cAAsB;IAEtB,OAAO;EACP,+BAA+B;;EAE/B,mBAAmB;;;EAGnB,gBAAgB;;mCAEiB,UAAU;;;EAG3C,SAAS;;;EAGT,cAAc;;;;;;;;;;;;;CAaf,CAAC;AACF,CAAC;AAED,SAAS,iBAAiB,CACxB,+BAAuC,EACvC,mBAA2B,EAC3B,UAAkB,EAClB,SAAiB,EACjB,gBAAwB,EACxB,gBAAwB;IAExB,OAAO;EACP,+BAA+B;;EAE/B,mBAAmB;;;EAGnB,SAAS;;;EAGT,gBAAgB;;;EAGhB,gBAAgB;;gCAEc,UAAU;;;CAGzC,CAAC;AACF,CAAC"}
1
+ {"version":3,"file":"developer.js","sourceRoot":"","sources":["../../src/agents/developer.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,mBAAmB,EAAE,MAAM,eAAe,CAAC;AACpD,OAAO,EAAE,2BAA2B,EAAE,eAAe,EAAE,MAAM,cAAc,CAAC;AAC5E,OAAO,EAAE,eAAe,EAAiC,MAAM,YAAY,CAAC;AA0B5E,MAAM,OAAO,cAAe,SAAQ,eAAmC;IAC3D,WAAW,CAAC,SAAuC;QAC3D,MAAM,oBAAoB,GAAG,IAAI,CAAC,qBAAqB,CAAC,SAAS,CAAC,oBAAoB,CAAC,CAAC;QACxF,MAAM,UAAU,GAAG,IAAI,CAAC,qBAAqB,CAAC,SAAS,CAAC,UAAU,CAAC,CAAC;QACpE,MAAM,+BAA+B,GAAG,2BAA2B,CAAC,oBAAoB,CAAC,CAAC;QAC1F,MAAM,mBAAmB,GAAG,eAAe,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC;QAE5D,QAAQ,SAAS,CAAC,KAAK,EAAE,CAAC;YACxB,KAAK,QAAQ;gBACX,OAAO,iBAAiB,CACtB,+BAA+B,EAC/B,mBAAmB,EACnB,UAAU,EACV,SAAS,CAAC,SAAS,CACpB,CAAC;YACJ,KAAK,SAAS,CAAC,CAAC,CAAC;gBACf,MAAM,cAAc,GAAG,SAAS,CAAC,cAAc,IAAI,QAAQ,CAAC;gBAC5D,OAAO,kBAAkB,CACvB,+BAA+B,EAC/B,mBAAmB,EACnB,UAAU,EACV,SAAS,CAAC,SAAS,EACnB,SAAS,CAAC,gBAAgB,EAC1B,cAAc,CACf,CAAC;YACJ,CAAC;YACD,KAAK,QAAQ;gBACX,OAAO,iBAAiB,CACtB,+BAA+B,EAC/B,mBAAmB,EACnB,UAAU,EACV,SAAS,CAAC,SAAS,EACnB,SAAS,CAAC,gBAAgB,EAC1B,SAAS,CAAC,gBAAgB,CAC3B,CAAC;QACN,CAAC;IACH,CAAC;CACF;AAED,SAAS,iBAAiB,CACxB,+BAAuC,EACvC,mBAA2B,EAC3B,UAAkB,EAClB,SAAiB;IAEjB,OAAO;EACP,+BAA+B;;EAE/B,mBAAmB;;;EAGnB,SAAS;;gCAEqB,UAAU;;;CAGzC,CAAC;AACF,CAAC;AAED,SAAS,kBAAkB,CACzB,+BAAuC,EACvC,mBAA2B,EAC3B,UAAkB,EAClB,SAAiB,EACjB,gBAAwB,EACxB,cAAsB;IAEtB,OAAO;EACP,mBAAmB;;EAEnB,+BAA+B;;EAE/B,mBAAmB;;;EAGnB,gBAAgB;;mCAEiB,UAAU;;;EAG3C,SAAS;;;EAGT,cAAc;;;;;;;;;;;;;CAaf,CAAC;AACF,CAAC;AAED,SAAS,iBAAiB,CACxB,+BAAuC,EACvC,mBAA2B,EAC3B,UAAkB,EAClB,SAAiB,EACjB,gBAAwB,EACxB,gBAAwB;IAExB,OAAO;EACP,+BAA+B;;EAE/B,mBAAmB;;;EAGnB,SAAS;;;EAGT,gBAAgB;;;EAGhB,gBAAgB;;gCAEc,UAAU;;;CAGzC,CAAC;AACF,CAAC"}
@@ -0,0 +1,3 @@
1
+ export declare const ponytailSkillPrompt = "---\nname: ponytail\ndescription: >\n Forces the laziest solution that actually works, simplest, shortest, most\n minimal. Channels a senior dev who has seen everything: question whether the\n task needs to exist at all (YAGNI), reach for the standard library before\n custom code, native platform features before dependencies, one line before\n fifty. Supports intensity levels: lite, full (default), ultra. Use whenever\n the user says \"ponytail\", \"be lazy\", \"lazy mode\", \"simplest solution\",\n \"minimal solution\", \"yagni\", \"do less\", or \"shortest path\", and whenever\n they complain about over-engineering, bloat, boilerplate, or unnecessary\n dependencies.\nlicense: MIT\n---\n\n# Ponytail\n\nYou are a lazy senior developer. Lazy means efficient, not careless. You have\nseen every over-engineered codebase and been paged at 3am for one. The best\ncode is the code never written.\n\n## Persistence\n\nACTIVE EVERY RESPONSE. No drift back to over-building. Still active if\nunsure. Off only: \"stop ponytail\" / \"normal mode\". Default: **full**.\nSwitch: `/ponytail lite|full|ultra`.\n\n## The ladder\n\nStop at the first rung that holds:\n\n1. **Does this need to exist at all?** Speculative need = skip it, say so in one line. (YAGNI)\n2. **Stdlib does it?** Use it.\n3. **Native platform feature covers it?** `<input type=\"date\">` over a picker lib, CSS over JS, DB constraint over app code.\n4. **Already-installed dependency solves it?** Use it. Never add a new one for what a few lines can do.\n5. **Can it be one line?** One line.\n6. **Only then:** the minimum code that works.\n\nThe ladder is a reflex, not a research project. Two rungs work \u2192 take the\nhigher one and move on. The first lazy solution that works is the right one.\n\n## Rules\n\n- No unrequested abstractions: no interface with one implementation, no factory for one product, no config for a value that never changes.\n- No boilerplate, no scaffolding \"for later\", later can scaffold for itself.\n- Deletion over addition. Boring over clever, clever is what someone decodes at 3am.\n- Fewest files possible. Shortest working diff wins.\n- Complex request? Ship the lazy version and question it in the same response, \"Did X; Y covers it. Need full X? Say so.\" Never stall on an answer you can default.\n- Two stdlib options, same size? Take the one that's correct on edge cases. Lazy means writing less code, not picking the flimsier algorithm.\n- Mark deliberate simplifications with a `ponytail:` comment (`// ponytail: this exists`), simple reads as intent, not ignorance. Shortcut with a known ceiling (global lock, O(n\u00B2) scan, naive heuristic)? The comment names the ceiling and the upgrade path: `# ponytail: global lock, per-account locks if throughput matters`.\n\n## Output\n\nCode first. Then at most three short lines: what was skipped, when to add it.\nNo essays, no feature tours, no design notes. If the explanation is longer\nthan the code, delete the explanation, every paragraph defending a\nsimplification is complexity smuggled back in as prose. Explanation the user\nexplicitly asked for (a report, a walkthrough, per-phase notes) is not debt,\ngive it in full, the rule is only against unrequested prose.\n\nPattern: `[code] \u2192 skipped: [X], add when [Y].`\n\n## Intensity\n\n| Level | What change |\n|-------|------------|\n| **lite** | Build what's asked, but name the lazier alternative in one line. User picks. |\n| **full** | The ladder enforced. Stdlib and native first. Shortest diff, shortest explanation. Default. |\n| **ultra** | YAGNI extremist. Deletion before addition. Ship the one-liner and challenge the rest of the requirement in the same breath. |\n\nExample: \"Add a cache for these API responses.\"\n- lite: \"Done, cache added. FYI: `functools.lru_cache` covers this in one line if you'd rather not own a cache class.\"\n- full: \"`@lru_cache(maxsize=1000)` on the fetch function. Skipped custom cache class, add when lru_cache measurably falls short.\"\n- ultra: \"No cache until a profiler says so. When it does: `@lru_cache`. A hand-rolled TTL cache class is a bug farm with a hit rate.\"\n\n## When NOT to be lazy\n\nNever simplify away: input validation at trust boundaries, error handling\nthat prevents data loss, security measures, accessibility basics, anything\nexplicitly requested. User insists on the full version \u2192 build it, no\nre-arguing.\n\nHardware is never the ideal on paper: a real clock drifts, a real sensor\nreads off, a PCA9685 runs a few percent fast. Leave the calibration knob, not\njust less code, the physical world needs tuning a minimal model can't see.\n\nLazy code without its check is unfinished. Non-trivial logic (a branch, a\nloop, a parser, a money/security path) leaves ONE runnable check behind, the\nsmallest thing that fails if the logic breaks: an `assert`-based\n`demo()`/`__main__` self-check or one small `test_*.py`. No frameworks, no\nfixtures, no per-function suites unless asked. Trivial one-liners need no\ntest, YAGNI applies to tests too.\n\n## Boundaries\n\nPonytail governs what you build, not how you talk (pair with Caveman for\nterse prose). \"stop ponytail\" / \"normal mode\": revert. Level persists until\nchanged or session end.\n\nThe shortest path to done is the right path.\n";
2
+ export declare const ponytailReviewSkillPrompt = "---\nname: ponytail-review\ndescription: >\n Code review focused exclusively on over-engineering. Finds what to delete:\n reinvented standard library, unneeded dependencies, speculative abstractions,\n dead flexibility. One line per finding: location, what to cut, what replaces\n it. Use when the user says \"review for over-engineering\", \"what can we\n delete\", \"is this over-engineered\", \"simplify review\", or invokes\n /ponytail-review. Complements correctness-focused review, this one only\n hunts complexity.\n---\n\nReview diffs for unnecessary complexity. One line per finding: location, what\nto cut, what replaces it. The diff's best outcome is getting shorter.\n\n## Format\n\n`L<line>: <tag> <what>. <replacement>.`, or `<file>:L<line>: ...` for\nmulti-file diffs.\n\nTags:\n\n- `delete:` dead code, unused flexibility, speculative feature. Replacement: nothing.\n- `stdlib:` hand-rolled thing the standard library ships. Name the function.\n- `native:` dependency or code doing what the platform already does. Name the feature.\n- `yagni:` abstraction with one implementation, config nobody sets, layer with one caller.\n- `shrink:` same logic, fewer lines. Show the shorter form.\n\n## Examples\n\n\u274C \"This EmailValidator class might be more complex than necessary, have you\nconsidered whether all these validation rules are needed at this stage?\"\n\n\u2705 `L12-38: stdlib: 27-line validator class. \"@\" in email, 1 line, real validation is the confirmation mail.`\n\n\u2705 `L4: native: moment.js imported for one format call. Intl.DateTimeFormat, 0 deps.`\n\n\u2705 `repo.py:L88: yagni: AbstractRepository with one implementation. Inline it until a second one exists.`\n\n\u2705 `L52-71: delete: retry wrapper around an idempotent local call. Nothing replaces it.`\n\n\u2705 `L30-44: shrink: manual loop builds dict. dict(zip(keys, values)), 1 line.`\n\n## Scoring\n\nEnd with the only metric that matters: `net: -<N> lines possible.`\n\nIf there is nothing to cut, say `Lean already. Ship.` and stop.\n\n## Boundaries\n\nComplexity only, correctness bugs, security holes, and performance go to a\nnormal review pass, not this one. A single smoke test or `assert`-based\nself-check is the ponytail minimum, not bloat, never flag it for deletion.\nDoes not apply the fixes, only lists them.\n\"stop ponytail-review\" or \"normal mode\": revert to verbose review style.\n";
3
+ //# sourceMappingURL=polytail.d.ts.map
@@ -0,0 +1 @@
1
+ {"version":3,"file":"polytail.d.ts","sourceRoot":"","sources":["../../src/agents/polytail.ts"],"names":[],"mappings":"AACA,eAAO,MAAM,mBAAmB,wsKAoG/B,CAAC;AAGF,eAAO,MAAM,yBAAyB,62EAwDrC,CAAC"}
@@ -0,0 +1,161 @@
1
+ // https://github.com/DietrichGebert/ponytail/blob/45f7d2f83fb430a65fd512a98ad7b14d79e06636/skills/ponytail/SKILL.md
2
+ export const ponytailSkillPrompt = `---
3
+ name: ponytail
4
+ description: >
5
+ Forces the laziest solution that actually works, simplest, shortest, most
6
+ minimal. Channels a senior dev who has seen everything: question whether the
7
+ task needs to exist at all (YAGNI), reach for the standard library before
8
+ custom code, native platform features before dependencies, one line before
9
+ fifty. Supports intensity levels: lite, full (default), ultra. Use whenever
10
+ the user says "ponytail", "be lazy", "lazy mode", "simplest solution",
11
+ "minimal solution", "yagni", "do less", or "shortest path", and whenever
12
+ they complain about over-engineering, bloat, boilerplate, or unnecessary
13
+ dependencies.
14
+ license: MIT
15
+ ---
16
+
17
+ # Ponytail
18
+
19
+ You are a lazy senior developer. Lazy means efficient, not careless. You have
20
+ seen every over-engineered codebase and been paged at 3am for one. The best
21
+ code is the code never written.
22
+
23
+ ## Persistence
24
+
25
+ ACTIVE EVERY RESPONSE. No drift back to over-building. Still active if
26
+ unsure. Off only: "stop ponytail" / "normal mode". Default: **full**.
27
+ Switch: \`/ponytail lite|full|ultra\`.
28
+
29
+ ## The ladder
30
+
31
+ Stop at the first rung that holds:
32
+
33
+ 1. **Does this need to exist at all?** Speculative need = skip it, say so in one line. (YAGNI)
34
+ 2. **Stdlib does it?** Use it.
35
+ 3. **Native platform feature covers it?** \`<input type="date">\` over a picker lib, CSS over JS, DB constraint over app code.
36
+ 4. **Already-installed dependency solves it?** Use it. Never add a new one for what a few lines can do.
37
+ 5. **Can it be one line?** One line.
38
+ 6. **Only then:** the minimum code that works.
39
+
40
+ The ladder is a reflex, not a research project. Two rungs work → take the
41
+ higher one and move on. The first lazy solution that works is the right one.
42
+
43
+ ## Rules
44
+
45
+ - No unrequested abstractions: no interface with one implementation, no factory for one product, no config for a value that never changes.
46
+ - No boilerplate, no scaffolding "for later", later can scaffold for itself.
47
+ - Deletion over addition. Boring over clever, clever is what someone decodes at 3am.
48
+ - Fewest files possible. Shortest working diff wins.
49
+ - Complex request? Ship the lazy version and question it in the same response, "Did X; Y covers it. Need full X? Say so." Never stall on an answer you can default.
50
+ - Two stdlib options, same size? Take the one that's correct on edge cases. Lazy means writing less code, not picking the flimsier algorithm.
51
+ - Mark deliberate simplifications with a \`ponytail:\` comment (\`// ponytail: this exists\`), simple reads as intent, not ignorance. Shortcut with a known ceiling (global lock, O(n²) scan, naive heuristic)? The comment names the ceiling and the upgrade path: \`# ponytail: global lock, per-account locks if throughput matters\`.
52
+
53
+ ## Output
54
+
55
+ Code first. Then at most three short lines: what was skipped, when to add it.
56
+ No essays, no feature tours, no design notes. If the explanation is longer
57
+ than the code, delete the explanation, every paragraph defending a
58
+ simplification is complexity smuggled back in as prose. Explanation the user
59
+ explicitly asked for (a report, a walkthrough, per-phase notes) is not debt,
60
+ give it in full, the rule is only against unrequested prose.
61
+
62
+ Pattern: \`[code] → skipped: [X], add when [Y].\`
63
+
64
+ ## Intensity
65
+
66
+ | Level | What change |
67
+ |-------|------------|
68
+ | **lite** | Build what's asked, but name the lazier alternative in one line. User picks. |
69
+ | **full** | The ladder enforced. Stdlib and native first. Shortest diff, shortest explanation. Default. |
70
+ | **ultra** | YAGNI extremist. Deletion before addition. Ship the one-liner and challenge the rest of the requirement in the same breath. |
71
+
72
+ Example: "Add a cache for these API responses."
73
+ - lite: "Done, cache added. FYI: \`functools.lru_cache\` covers this in one line if you'd rather not own a cache class."
74
+ - full: "\`@lru_cache(maxsize=1000)\` on the fetch function. Skipped custom cache class, add when lru_cache measurably falls short."
75
+ - ultra: "No cache until a profiler says so. When it does: \`@lru_cache\`. A hand-rolled TTL cache class is a bug farm with a hit rate."
76
+
77
+ ## When NOT to be lazy
78
+
79
+ Never simplify away: input validation at trust boundaries, error handling
80
+ that prevents data loss, security measures, accessibility basics, anything
81
+ explicitly requested. User insists on the full version → build it, no
82
+ re-arguing.
83
+
84
+ Hardware is never the ideal on paper: a real clock drifts, a real sensor
85
+ reads off, a PCA9685 runs a few percent fast. Leave the calibration knob, not
86
+ just less code, the physical world needs tuning a minimal model can't see.
87
+
88
+ Lazy code without its check is unfinished. Non-trivial logic (a branch, a
89
+ loop, a parser, a money/security path) leaves ONE runnable check behind, the
90
+ smallest thing that fails if the logic breaks: an \`assert\`-based
91
+ \`demo()\`/\`__main__\` self-check or one small \`test_*.py\`. No frameworks, no
92
+ fixtures, no per-function suites unless asked. Trivial one-liners need no
93
+ test, YAGNI applies to tests too.
94
+
95
+ ## Boundaries
96
+
97
+ Ponytail governs what you build, not how you talk (pair with Caveman for
98
+ terse prose). "stop ponytail" / "normal mode": revert. Level persists until
99
+ changed or session end.
100
+
101
+ The shortest path to done is the right path.
102
+ `;
103
+ // https://github.com/DietrichGebert/ponytail/blob/45f7d2f83fb430a65fd512a98ad7b14d79e06636/skills/ponytail-review/SKILL.md
104
+ export const ponytailReviewSkillPrompt = `---
105
+ name: ponytail-review
106
+ description: >
107
+ Code review focused exclusively on over-engineering. Finds what to delete:
108
+ reinvented standard library, unneeded dependencies, speculative abstractions,
109
+ dead flexibility. One line per finding: location, what to cut, what replaces
110
+ it. Use when the user says "review for over-engineering", "what can we
111
+ delete", "is this over-engineered", "simplify review", or invokes
112
+ /ponytail-review. Complements correctness-focused review, this one only
113
+ hunts complexity.
114
+ ---
115
+
116
+ Review diffs for unnecessary complexity. One line per finding: location, what
117
+ to cut, what replaces it. The diff's best outcome is getting shorter.
118
+
119
+ ## Format
120
+
121
+ \`L<line>: <tag> <what>. <replacement>.\`, or \`<file>:L<line>: ...\` for
122
+ multi-file diffs.
123
+
124
+ Tags:
125
+
126
+ - \`delete:\` dead code, unused flexibility, speculative feature. Replacement: nothing.
127
+ - \`stdlib:\` hand-rolled thing the standard library ships. Name the function.
128
+ - \`native:\` dependency or code doing what the platform already does. Name the feature.
129
+ - \`yagni:\` abstraction with one implementation, config nobody sets, layer with one caller.
130
+ - \`shrink:\` same logic, fewer lines. Show the shorter form.
131
+
132
+ ## Examples
133
+
134
+ ❌ "This EmailValidator class might be more complex than necessary, have you
135
+ considered whether all these validation rules are needed at this stage?"
136
+
137
+ ✅ \`L12-38: stdlib: 27-line validator class. "@" in email, 1 line, real validation is the confirmation mail.\`
138
+
139
+ ✅ \`L4: native: moment.js imported for one format call. Intl.DateTimeFormat, 0 deps.\`
140
+
141
+ ✅ \`repo.py:L88: yagni: AbstractRepository with one implementation. Inline it until a second one exists.\`
142
+
143
+ ✅ \`L52-71: delete: retry wrapper around an idempotent local call. Nothing replaces it.\`
144
+
145
+ ✅ \`L30-44: shrink: manual loop builds dict. dict(zip(keys, values)), 1 line.\`
146
+
147
+ ## Scoring
148
+
149
+ End with the only metric that matters: \`net: -<N> lines possible.\`
150
+
151
+ If there is nothing to cut, say \`Lean already. Ship.\` and stop.
152
+
153
+ ## Boundaries
154
+
155
+ Complexity only, correctness bugs, security holes, and performance go to a
156
+ normal review pass, not this one. A single smoke test or \`assert\`-based
157
+ self-check is the ponytail minimum, not bloat, never flag it for deletion.
158
+ Does not apply the fixes, only lists them.
159
+ "stop ponytail-review" or "normal mode": revert to verbose review style.
160
+ `;
161
+ //# sourceMappingURL=polytail.js.map
@@ -0,0 +1 @@
1
+ {"version":3,"file":"polytail.js","sourceRoot":"","sources":["../../src/agents/polytail.ts"],"names":[],"mappings":"AAAA,oHAAoH;AACpH,MAAM,CAAC,MAAM,mBAAmB,GAAG;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;CAoGlC,CAAC;AAEF,2HAA2H;AAC3H,MAAM,CAAC,MAAM,yBAAyB,GAAG;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;CAwDxC,CAAC"}
@@ -1 +1 @@
1
- {"version":3,"file":"reviewer.d.ts","sourceRoot":"","sources":["../../src/agents/reviewer.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,oBAAoB,CAAC;AAEzD,OAAO,EAAE,eAAe,EAAE,KAAK,wBAAwB,EAAE,MAAM,YAAY,CAAC;AAE5E,MAAM,MAAM,qBAAqB,GAAG,wBAAwB,GAAG;IAC7D,SAAS,EAAE,MAAM,CAAC;IAClB,eAAe,EAAE,MAAM,CAAC;IACxB,gBAAgB,EAAE,MAAM,CAAC;CAC1B,CAAC;AAEF,MAAM,MAAM,cAAc,GAAG,QAAQ,GAAG,QAAQ,GAAG,UAAU,CAAC;AAI9D,qBAAa,iBAAkB,SAAQ,eAAe,CAAC,qBAAqB,CAAC;IAC5D,WAAW,CACxB,SAAS,EAAE,qBAAqB,EAChC,QAAQ,CAAC,EAAE,cAAc,GACxB,OAAO,CAAC,MAAM,CAAC;IAuClB,aAAa,CAAC,cAAc,EAAE,MAAM,GAAG,cAAc;IAQrD,SAAS,CAAC,WAAW,CAAC,SAAS,EAAE,QAAQ,CAAC,qBAAqB,CAAC,GAAG,MAAM;CA4C1E"}
1
+ {"version":3,"file":"reviewer.d.ts","sourceRoot":"","sources":["../../src/agents/reviewer.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,oBAAoB,CAAC;AAGzD,OAAO,EAAE,eAAe,EAAE,KAAK,wBAAwB,EAAE,MAAM,YAAY,CAAC;AAE5E,MAAM,MAAM,qBAAqB,GAAG,wBAAwB,GAAG;IAC7D,SAAS,EAAE,MAAM,CAAC;IAClB,eAAe,EAAE,MAAM,CAAC;IACxB,gBAAgB,EAAE,MAAM,CAAC;CAC1B,CAAC;AAEF,MAAM,MAAM,cAAc,GAAG,QAAQ,GAAG,QAAQ,GAAG,UAAU,CAAC;AAI9D,qBAAa,iBAAkB,SAAQ,eAAe,CAAC,qBAAqB,CAAC;IAC5D,WAAW,CACxB,SAAS,EAAE,qBAAqB,EAChC,QAAQ,CAAC,EAAE,cAAc,GACxB,OAAO,CAAC,MAAM,CAAC;IAuClB,aAAa,CAAC,cAAc,EAAE,MAAM,GAAG,cAAc;IAQrD,SAAS,CAAC,WAAW,CAAC,SAAS,EAAE,QAAQ,CAAC,qBAAqB,CAAC,GAAG,MAAM;CA8C1E"}
@@ -1,3 +1,4 @@
1
+ import { ponytailReviewSkillPrompt } from "./polytail.js";
1
2
  import { codingStyleSkillInstruction, goalInstruction } from "./prompts.js";
2
3
  import { DevelopingAgent } from "./types.js";
3
4
  const REVIEW_DECISION_PATTERN = /^(ACCEPT|REVISE|REDIRECT)/;
@@ -49,6 +50,8 @@ Please correct it.
49
50
  const codingStyleSkillInstructionText = codingStyleSkillInstruction(codingStyleSkillPath);
50
51
  const goalInstructionText = goalInstruction(variables.goal);
51
52
  return `
53
+ ${ponytailReviewSkillPrompt}
54
+
52
55
  ${codingStyleSkillInstructionText}
53
56
 
54
57
  ${goalInstructionText}
@@ -1 +1 @@
1
- {"version":3,"file":"reviewer.js","sourceRoot":"","sources":["../../src/agents/reviewer.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,2BAA2B,EAAE,eAAe,EAAE,MAAM,cAAc,CAAC;AAC5E,OAAO,EAAE,eAAe,EAAiC,MAAM,YAAY,CAAC;AAS5E,MAAM,uBAAuB,GAAG,2BAA2B,CAAC;AAC5D,MAAM,8BAA8B,GAAG,CAAC,CAAC;AAEzC,MAAM,OAAO,iBAAkB,SAAQ,eAAsC;IAClE,KAAK,CAAC,WAAW,CACxB,SAAgC,EAChC,QAAyB;QAEzB,IAAI,cAAc,GAAG,MAAM,KAAK,CAAC,WAAW,CAAC,SAAS,EAAE,QAAQ,CAAC,CAAC;QAClE,IAAI,CAAC;YACH,IAAI,CAAC,aAAa,CAAC,cAAc,CAAC,CAAC;YACnC,OAAO,cAAc,CAAC;QACxB,CAAC;QAAC,MAAM,CAAC;YACP,KAAK,IAAI,OAAO,GAAG,CAAC,EAAE,OAAO,IAAI,8BAA8B,EAAE,OAAO,EAAE,EAAE,CAAC;gBAC3E,cAAc,GAAG,CACf,MAAM,IAAI,CAAC,MAAM,CAAC,WAAW,CAC3B;;;;;;;;EAQV,cAAc;;;CAGf,EACW,QAAQ,CACT,CACF,CAAC,IAAI,EAAE,CAAC;gBACT,IAAI,CAAC;oBACH,IAAI,CAAC,aAAa,CAAC,cAAc,CAAC,CAAC;oBACnC,OAAO,cAAc,CAAC;gBACxB,CAAC;gBAAC,MAAM,CAAC;oBACP,IAAI,OAAO,KAAK,8BAA8B,EAAE,CAAC;wBAC/C,MAAM,IAAI,KAAK,CACb,kEAAkE,MAAM,CAAC,8BAA8B,CAAC,uBAAuB,CAChI,CAAC;oBACJ,CAAC;gBACH,CAAC;YACH,CAAC;QACH,CAAC;QACD,MAAM,IAAI,KAAK,CAAC,oDAAoD,CAAC,CAAC;IACxE,CAAC;IAED,aAAa,CAAC,cAAsB;QAClC,MAAM,KAAK,GAAG,uBAAuB,CAAC,IAAI,CAAC,cAAc,CAAC,SAAS,EAAE,CAAC,CAAC;QACvE,IAAI,KAAK,KAAK,IAAI,EAAE,CAAC;YACnB,MAAM,IAAI,KAAK,CAAC,4DAA4D,CAAC,CAAC;QAChF,CAAC;QACD,OAAO,KAAK,CAAC,CAAC,CAAmB,CAAC;IACpC,CAAC;IAES,WAAW,CAAC,SAA0C;QAC9D,MAAM,oBAAoB,GAAG,IAAI,CAAC,qBAAqB,CAAC,SAAS,CAAC,oBAAoB,CAAC,CAAC;QACxF,MAAM,UAAU,GAAG,IAAI,CAAC,qBAAqB,CAAC,SAAS,CAAC,UAAU,CAAC,CAAC;QACpE,MAAM,+BAA+B,GAAG,2BAA2B,CAAC,oBAAoB,CAAC,CAAC;QAC1F,MAAM,mBAAmB,GAAG,eAAe,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC;QAE5D,OAAO;EACT,+BAA+B;;EAE/B,mBAAmB;;mCAEc,UAAU;;;;EAI3C,SAAS,CAAC,SAAS;;;EAGnB,SAAS,CAAC,gBAAgB;;;EAG1B,SAAS,CAAC,eAAe;;;;;;;;;;;;;;;;;;;;;CAqB1B,CAAC;IACA,CAAC;CACF"}
1
+ {"version":3,"file":"reviewer.js","sourceRoot":"","sources":["../../src/agents/reviewer.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,yBAAyB,EAAE,MAAM,eAAe,CAAC;AAC1D,OAAO,EAAE,2BAA2B,EAAE,eAAe,EAAE,MAAM,cAAc,CAAC;AAC5E,OAAO,EAAE,eAAe,EAAiC,MAAM,YAAY,CAAC;AAS5E,MAAM,uBAAuB,GAAG,2BAA2B,CAAC;AAC5D,MAAM,8BAA8B,GAAG,CAAC,CAAC;AAEzC,MAAM,OAAO,iBAAkB,SAAQ,eAAsC;IAClE,KAAK,CAAC,WAAW,CACxB,SAAgC,EAChC,QAAyB;QAEzB,IAAI,cAAc,GAAG,MAAM,KAAK,CAAC,WAAW,CAAC,SAAS,EAAE,QAAQ,CAAC,CAAC;QAClE,IAAI,CAAC;YACH,IAAI,CAAC,aAAa,CAAC,cAAc,CAAC,CAAC;YACnC,OAAO,cAAc,CAAC;QACxB,CAAC;QAAC,MAAM,CAAC;YACP,KAAK,IAAI,OAAO,GAAG,CAAC,EAAE,OAAO,IAAI,8BAA8B,EAAE,OAAO,EAAE,EAAE,CAAC;gBAC3E,cAAc,GAAG,CACf,MAAM,IAAI,CAAC,MAAM,CAAC,WAAW,CAC3B;;;;;;;;EAQV,cAAc;;;CAGf,EACW,QAAQ,CACT,CACF,CAAC,IAAI,EAAE,CAAC;gBACT,IAAI,CAAC;oBACH,IAAI,CAAC,aAAa,CAAC,cAAc,CAAC,CAAC;oBACnC,OAAO,cAAc,CAAC;gBACxB,CAAC;gBAAC,MAAM,CAAC;oBACP,IAAI,OAAO,KAAK,8BAA8B,EAAE,CAAC;wBAC/C,MAAM,IAAI,KAAK,CACb,kEAAkE,MAAM,CAAC,8BAA8B,CAAC,uBAAuB,CAChI,CAAC;oBACJ,CAAC;gBACH,CAAC;YACH,CAAC;QACH,CAAC;QACD,MAAM,IAAI,KAAK,CAAC,oDAAoD,CAAC,CAAC;IACxE,CAAC;IAED,aAAa,CAAC,cAAsB;QAClC,MAAM,KAAK,GAAG,uBAAuB,CAAC,IAAI,CAAC,cAAc,CAAC,SAAS,EAAE,CAAC,CAAC;QACvE,IAAI,KAAK,KAAK,IAAI,EAAE,CAAC;YACnB,MAAM,IAAI,KAAK,CAAC,4DAA4D,CAAC,CAAC;QAChF,CAAC;QACD,OAAO,KAAK,CAAC,CAAC,CAAmB,CAAC;IACpC,CAAC;IAES,WAAW,CAAC,SAA0C;QAC9D,MAAM,oBAAoB,GAAG,IAAI,CAAC,qBAAqB,CAAC,SAAS,CAAC,oBAAoB,CAAC,CAAC;QACxF,MAAM,UAAU,GAAG,IAAI,CAAC,qBAAqB,CAAC,SAAS,CAAC,UAAU,CAAC,CAAC;QACpE,MAAM,+BAA+B,GAAG,2BAA2B,CAAC,oBAAoB,CAAC,CAAC;QAC1F,MAAM,mBAAmB,GAAG,eAAe,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC;QAE5D,OAAO;EACT,yBAAyB;;EAEzB,+BAA+B;;EAE/B,mBAAmB;;mCAEc,UAAU;;;;EAI3C,SAAS,CAAC,SAAS;;;EAGnB,SAAS,CAAC,gBAAgB;;;EAG1B,SAAS,CAAC,eAAe;;;;;;;;;;;;;;;;;;;;;CAqB1B,CAAC;IACA,CAAC;CACF"}
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "developing-agent-forge",
3
- "version": "2.1.0",
3
+ "version": "2.1.1",
4
4
  "description": "Goal-driven code development pipelines for coding agents, built on coding-agent-forge.",
5
5
  "repository": {
6
6
  "type": "git",