factory-factory 0.2.0 → 0.2.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.
- package/dist/client/assets/{_basePickBy-BUCmhYMi.js → _basePickBy-DikkcLWt.js} +1 -1
- package/dist/client/assets/{_baseUniq-DkCvE4O6.js → _baseUniq-DXQLkszL.js} +1 -1
- package/dist/client/assets/{arc-C1SZ3gFz.js → arc-PlxYF4gs.js} +1 -1
- package/dist/client/assets/{architectureDiagram-VXUJARFQ-DJw9fXI5.js → architectureDiagram-VXUJARFQ-C15kzB-a.js} +1 -1
- package/dist/client/assets/{blockDiagram-VD42YOAC-DxNjgXL1.js → blockDiagram-VD42YOAC-CpsrSECi.js} +1 -1
- package/dist/client/assets/{c4Diagram-YG6GDRKO-DYyf8hqJ.js → c4Diagram-YG6GDRKO-lUG_fBjc.js} +1 -1
- package/dist/client/assets/channel-DibAzDxU.js +1 -0
- package/dist/client/assets/{chunk-4BX2VUAB-D_xs3w4g.js → chunk-4BX2VUAB-DL5SmHl8.js} +1 -1
- package/dist/client/assets/{chunk-55IACEB6-NeVyGiBw.js → chunk-55IACEB6-6RonniJP.js} +1 -1
- package/dist/client/assets/{chunk-B4BG7PRW-EyCx-NZJ.js → chunk-B4BG7PRW-BurHg0kj.js} +1 -1
- package/dist/client/assets/{chunk-DI55MBZ5-PoOxaYU4.js → chunk-DI55MBZ5-BJ01eABM.js} +1 -1
- package/dist/client/assets/{chunk-FMBD7UC4-D1FKmcwY.js → chunk-FMBD7UC4-BnvMi7Ay.js} +1 -1
- package/dist/client/assets/{chunk-QN33PNHL-B-nhuti6.js → chunk-QN33PNHL-CHzCvA9V.js} +1 -1
- package/dist/client/assets/{chunk-QZHKN3VN-BvwrXwAL.js → chunk-QZHKN3VN-BYtef6TB.js} +1 -1
- package/dist/client/assets/{chunk-TZMSLE5B-CwcAJq0Y.js → chunk-TZMSLE5B-5jRzcEei.js} +1 -1
- package/dist/client/assets/classDiagram-2ON5EDUG-moaJKwcI.js +1 -0
- package/dist/client/assets/classDiagram-v2-WZHVMYZB-moaJKwcI.js +1 -0
- package/dist/client/assets/clone-Cl_PwNgC.js +1 -0
- package/dist/client/assets/{cose-bilkent-S5V4N54A-70G8lj7y.js → cose-bilkent-S5V4N54A-BRiIkmzf.js} +1 -1
- package/dist/client/assets/{dagre-6UL2VRFP-CxTqTZJX.js → dagre-6UL2VRFP-DB-glvK1.js} +1 -1
- package/dist/client/assets/{diagram-PSM6KHXK-wdA5gBhP.js → diagram-PSM6KHXK-CivlU3GF.js} +1 -1
- package/dist/client/assets/{diagram-QEK2KX5R-BWiPIEBn.js → diagram-QEK2KX5R-ux8xYm0V.js} +1 -1
- package/dist/client/assets/{diagram-S2PKOQOG-CzhUSsIe.js → diagram-S2PKOQOG-BQ2G1KUX.js} +1 -1
- package/dist/client/assets/{erDiagram-Q2GNP2WA-DvowAcu-.js → erDiagram-Q2GNP2WA-pDV9J51h.js} +1 -1
- package/dist/client/assets/{flowDiagram-NV44I4VS-B9jMwSHS.js → flowDiagram-NV44I4VS-BzDPdmg2.js} +1 -1
- package/dist/client/assets/{ganttDiagram-JELNMOA3-Bfaxm8Qf.js → ganttDiagram-JELNMOA3-asSHWQ1i.js} +1 -1
- package/dist/client/assets/{gitGraphDiagram-NY62KEGX-5EWbjg13.js → gitGraphDiagram-NY62KEGX-Duv9FC3a.js} +1 -1
- package/dist/client/assets/{graph-Bms-Sm3d.js → graph-6ULGC9RP.js} +1 -1
- package/dist/client/assets/{index-Cp1e-xYL.js → index-DTaPBU72.js} +100 -100
- package/dist/client/assets/{infoDiagram-WHAUD3N6-DYAoqgKw.js → infoDiagram-WHAUD3N6-BXQPO9ey.js} +1 -1
- package/dist/client/assets/{journeyDiagram-XKPGCS4Q-B0HifI98.js → journeyDiagram-XKPGCS4Q-BTZFCnxJ.js} +1 -1
- package/dist/client/assets/{kanban-definition-3W4ZIXB7-BnBdOX8w.js → kanban-definition-3W4ZIXB7-BgM5-joq.js} +1 -1
- package/dist/client/assets/{layout-Tzmamt4J.js → layout-upXVONoQ.js} +1 -1
- package/dist/client/assets/{linear-DuZ88VjU.js → linear-DSEaz9ry.js} +1 -1
- package/dist/client/assets/{mindmap-definition-VGOIOE7T-mIqXf7ud.js → mindmap-definition-VGOIOE7T-BWg5D2xt.js} +1 -1
- package/dist/client/assets/{pieDiagram-ADFJNKIX-LU3IJ_3z.js → pieDiagram-ADFJNKIX-DiebpcAz.js} +1 -1
- package/dist/client/assets/{quadrantDiagram-AYHSOK5B-RcfMXzE1.js → quadrantDiagram-AYHSOK5B-CRdJB1rK.js} +1 -1
- package/dist/client/assets/{requirementDiagram-UZGBJVZJ-fCa8OVB7.js → requirementDiagram-UZGBJVZJ-BWWp8mm2.js} +1 -1
- package/dist/client/assets/{sankeyDiagram-TZEHDZUN-ByI3z-Yp.js → sankeyDiagram-TZEHDZUN-Be_z4SEy.js} +1 -1
- package/dist/client/assets/{sequenceDiagram-WL72ISMW-Bi2fH0fZ.js → sequenceDiagram-WL72ISMW-Cmh_NUjH.js} +1 -1
- package/dist/client/assets/{stateDiagram-FKZM4ZOC-BcKHGHcW.js → stateDiagram-FKZM4ZOC-CMVA9mym.js} +1 -1
- package/dist/client/assets/stateDiagram-v2-4FDKWEC3-CELclHRO.js +1 -0
- package/dist/client/assets/{terminal-instance-CXvzjFQW.js → terminal-instance-CihJB_Qi.js} +1 -1
- package/dist/client/assets/{timeline-definition-IT6M3QCI-7U7VPI2C.js → timeline-definition-IT6M3QCI-CkTun4nl.js} +1 -1
- package/dist/client/assets/{treemap-KMMF4GRG-B1RMkeMx.js → treemap-KMMF4GRG-EA1NDtbB.js} +1 -1
- package/dist/client/assets/{xychartDiagram-PRI3JC2R-DC1DrOy8.js → xychartDiagram-PRI3JC2R-DZ9QOnNM.js} +1 -1
- package/dist/client/index.html +1 -1
- package/dist/prompts/workflows/pr-review-fix.md +32 -44
- package/dist/prompts/workflows/ratchet.md +13 -9
- package/dist/src/backend/agents/process-adapter.d.ts +2 -2
- package/dist/src/backend/agents/process-adapter.d.ts.map +1 -1
- package/dist/src/backend/agents/process-adapter.js +2 -2
- package/dist/src/backend/agents/process-adapter.js.map +1 -1
- package/dist/src/backend/claude/index.d.ts +1 -1
- package/dist/src/backend/claude/index.d.ts.map +1 -1
- package/dist/src/backend/claude/index.js +2 -2
- package/dist/src/backend/claude/index.js.map +1 -1
- package/dist/src/backend/claude/monitoring.js +1 -1
- package/dist/src/backend/claude/monitoring.js.map +1 -1
- package/dist/src/backend/interceptors/conversation-rename.interceptor.d.ts.map +1 -1
- package/dist/src/backend/interceptors/conversation-rename.interceptor.js +6 -1
- package/dist/src/backend/interceptors/conversation-rename.interceptor.js.map +1 -1
- package/dist/src/backend/services/chat-message-handlers/handlers/handlers.test.js +1 -1
- package/dist/src/backend/services/chat-message-handlers/handlers/handlers.test.js.map +1 -1
- package/dist/src/backend/services/chat-message-handlers/handlers/user-input.handler.d.ts.map +1 -1
- package/dist/src/backend/services/chat-message-handlers/handlers/user-input.handler.js +5 -1
- package/dist/src/backend/services/chat-message-handlers/handlers/user-input.handler.js.map +1 -1
- package/dist/src/backend/services/chat-message-handlers.service.js +1 -1
- package/dist/src/backend/services/chat-message-handlers.service.js.map +1 -1
- package/dist/src/backend/services/ci-fixer.service.d.ts.map +1 -1
- package/dist/src/backend/services/ci-fixer.service.js +12 -2
- package/dist/src/backend/services/ci-fixer.service.js.map +1 -1
- package/dist/src/backend/services/ci-fixer.service.test.js +3 -3
- package/dist/src/backend/services/ci-fixer.service.test.js.map +1 -1
- package/dist/src/backend/services/ci-monitor.service.d.ts.map +1 -1
- package/dist/src/backend/services/ci-monitor.service.js +3 -1
- package/dist/src/backend/services/ci-monitor.service.js.map +1 -1
- package/dist/src/backend/services/github-cli.service.d.ts +5 -0
- package/dist/src/backend/services/github-cli.service.d.ts.map +1 -1
- package/dist/src/backend/services/github-cli.service.js +47 -28
- package/dist/src/backend/services/github-cli.service.js.map +1 -1
- package/dist/src/backend/services/github-cli.service.test.d.ts +2 -0
- package/dist/src/backend/services/github-cli.service.test.d.ts.map +1 -0
- package/dist/src/backend/services/github-cli.service.test.js +302 -0
- package/dist/src/backend/services/github-cli.service.test.js.map +1 -0
- package/dist/src/backend/services/pr-review-fixer.service.d.ts.map +1 -1
- package/dist/src/backend/services/pr-review-fixer.service.js +12 -2
- package/dist/src/backend/services/pr-review-fixer.service.js.map +1 -1
- package/dist/src/backend/services/ratchet.service.d.ts.map +1 -1
- package/dist/src/backend/services/ratchet.service.js +55 -26
- package/dist/src/backend/services/ratchet.service.js.map +1 -1
- package/dist/src/backend/services/worktree-lifecycle.service.d.ts.map +1 -1
- package/dist/src/backend/services/worktree-lifecycle.service.js +16 -2
- package/dist/src/backend/services/worktree-lifecycle.service.js.map +1 -1
- package/dist/tsconfig.backend.tsbuildinfo +1 -1
- package/package.json +1 -1
- package/prompts/workflows/pr-review-fix.md +32 -44
- package/prompts/workflows/ratchet.md +13 -9
- package/dist/client/assets/channel-C1q-YHIv.js +0 -1
- package/dist/client/assets/classDiagram-2ON5EDUG-BRch7PJo.js +0 -1
- package/dist/client/assets/classDiagram-v2-WZHVMYZB-BRch7PJo.js +0 -1
- package/dist/client/assets/clone-D8y6D-S6.js +0 -1
- package/dist/client/assets/stateDiagram-v2-4FDKWEC3-DIADSwH3.js +0 -1
package/package.json
CHANGED
|
@@ -8,54 +8,42 @@ expectsPR: false
|
|
|
8
8
|
|
|
9
9
|
You are addressing PR review comments for this workspace's pull request. Your goal is to implement the requested changes and satisfy the reviewers.
|
|
10
10
|
|
|
11
|
-
|
|
12
|
-
|
|
13
|
-
### 1. Understand the Feedback
|
|
14
|
-
- Review the comments provided in the initial message carefully
|
|
15
|
-
- Pay attention to both inline code comments and general review comments
|
|
16
|
-
- Understand what changes each reviewer is requesting
|
|
17
|
-
- Note any questions or clarifications the reviewer is asking for
|
|
18
|
-
|
|
19
|
-
### 2. Plan Your Changes
|
|
20
|
-
- Identify which files need to be modified
|
|
21
|
-
- Determine the scope of changes needed
|
|
22
|
-
- Consider if the requested change impacts other parts of the codebase
|
|
23
|
-
- If a request is unclear, make a reasonable interpretation and document your assumptions
|
|
24
|
-
|
|
25
|
-
### 3. Implement the Changes
|
|
26
|
-
- Address each review comment systematically
|
|
27
|
-
- Make focused changes that directly address the feedback
|
|
28
|
-
- Avoid scope creep - only fix what was requested
|
|
29
|
-
- If you disagree with a suggestion, implement it anyway but note your reasoning in the commit message
|
|
11
|
+
**CRITICAL: Execute autonomously. Do not ask the user for input, confirmation, or clarification. Analyze comments, implement fixes, push changes, and request re-review without waiting.**
|
|
30
12
|
|
|
31
|
-
|
|
32
|
-
- Run the test suite: `pnpm test`
|
|
33
|
-
- Run type checking: `pnpm typecheck`
|
|
34
|
-
- Run linting: `pnpm check:fix`
|
|
35
|
-
- Test any affected functionality manually if appropriate
|
|
13
|
+
## Workflow
|
|
36
14
|
|
|
37
|
-
###
|
|
38
|
-
-
|
|
39
|
-
-
|
|
40
|
-
-
|
|
15
|
+
### 1. Analyze Comments
|
|
16
|
+
- Review all comments in the initial message
|
|
17
|
+
- Categorize each as **actionable** (requires code changes) or **informational** (no action needed)
|
|
18
|
+
- Informational comments include: automated coverage reports, CI status notifications, or comments that just acknowledge/thank without requesting changes
|
|
19
|
+
- For actionable comments, determine what changes are needed
|
|
41
20
|
|
|
42
|
-
###
|
|
43
|
-
-
|
|
44
|
-
-
|
|
21
|
+
### 2. Implement Fixes
|
|
22
|
+
- Address each actionable comment systematically
|
|
23
|
+
- Make focused changes that directly address the feedback
|
|
24
|
+
- If a request is unclear, make a reasonable interpretation and proceed
|
|
25
|
+
- Do not ask for clarification - make your best judgment
|
|
45
26
|
|
|
46
|
-
|
|
27
|
+
### 3. Verify Changes
|
|
28
|
+
\`\`\`bash
|
|
29
|
+
pnpm test && pnpm typecheck && pnpm check:fix
|
|
30
|
+
\`\`\`
|
|
47
31
|
|
|
48
|
-
|
|
49
|
-
|
|
50
|
-
-
|
|
51
|
-
|
|
52
|
-
- **Document decisions**: If you make judgment calls, explain your reasoning
|
|
32
|
+
### 4. Commit and Push
|
|
33
|
+
\`\`\`bash
|
|
34
|
+
git add -A && git commit -m "Address review comments" && git push
|
|
35
|
+
\`\`\`
|
|
53
36
|
|
|
54
|
-
|
|
37
|
+
### 5. Request Re-review
|
|
38
|
+
Post a comment mentioning the reviewers. Use `gh pr list --head $(git branch --show-current)` to find the PR number if not already known:
|
|
39
|
+
\`\`\`bash
|
|
40
|
+
gh pr comment $(gh pr list --head $(git branch --show-current) --json number --jq '.[0].number') --body "@reviewer1 @reviewer2 I've addressed the review comments. Please re-review when you have a chance."
|
|
41
|
+
\`\`\`
|
|
42
|
+
|
|
43
|
+
## Guidelines
|
|
55
44
|
|
|
56
|
-
- **
|
|
57
|
-
- **
|
|
58
|
-
- **
|
|
59
|
-
- **
|
|
60
|
-
- **
|
|
61
|
-
- **Logic changes**: Implement the alternative approach suggested
|
|
45
|
+
- **Operate autonomously**: Never ask the user what to do. Analyze, decide, implement, push.
|
|
46
|
+
- **Address all actionable comments**: Skip only truly informational comments
|
|
47
|
+
- **Keep changes focused**: Only change what was requested
|
|
48
|
+
- **Test before pushing**: Always run local checks first
|
|
49
|
+
- **Handle bot comments**: Analyze bot comments (coverage, CI) for actionable items. If purely informational, move on without code changes.
|
|
@@ -8,9 +8,11 @@ expectsPR: false
|
|
|
8
8
|
|
|
9
9
|
You are the Ratchet agent, responsible for automatically progressing this PR toward merge. You will receive notifications about different issues (CI failures, merge conflicts, review comments) and should address them systematically.
|
|
10
10
|
|
|
11
|
+
**CRITICAL: You must operate autonomously. Do not ask the user for input, confirmation, or clarification. Analyze the situation, make decisions, implement fixes, and push changes without waiting for approval.**
|
|
12
|
+
|
|
11
13
|
## Your Role
|
|
12
14
|
|
|
13
|
-
The Ratchet system monitors PRs and notifies you when action is needed. You
|
|
15
|
+
The Ratchet system monitors PRs and notifies you when action is needed. You must:
|
|
14
16
|
|
|
15
17
|
1. **Fix CI failures** - Investigate and fix broken tests, type errors, or lint issues
|
|
16
18
|
2. **Resolve merge conflicts** - Merge the base branch and resolve any conflicts
|
|
@@ -66,18 +68,20 @@ When notified of merge conflicts:
|
|
|
66
68
|
|
|
67
69
|
## Review Comment Resolution
|
|
68
70
|
|
|
69
|
-
When notified of review comments:
|
|
71
|
+
When notified of review comments, execute autonomously:
|
|
72
|
+
|
|
73
|
+
1. **Analyze each comment** - Determine if actionable. Informational comments (e.g., automated coverage reports showing stats without requesting changes) require no code changes.
|
|
74
|
+
2. **Implement fixes** - Address each actionable comment directly. Do not ask for clarification.
|
|
75
|
+
3. **Verify**: `pnpm test && pnpm typecheck && pnpm check:fix`
|
|
76
|
+
4. **Commit and push**: `git add -A && git commit -m "Address review comments" && git push`
|
|
77
|
+
5. **Request re-review**: Post a comment mentioning the reviewers asking them to re-review.
|
|
70
78
|
|
|
71
|
-
|
|
72
|
-
2. Plan the necessary changes
|
|
73
|
-
3. Implement changes to address feedback
|
|
74
|
-
4. Verify with tests
|
|
75
|
-
5. Commit and push
|
|
76
|
-
6. Request re-review from the reviewers
|
|
79
|
+
**Do not ask the user what to do. Make reasonable decisions and proceed.**
|
|
77
80
|
|
|
78
81
|
## Guidelines
|
|
79
82
|
|
|
83
|
+
- **Operate autonomously**: Never ask the user for input or confirmation. Make decisions and execute.
|
|
80
84
|
- **Focus on the current issue**: Don't refactor or improve unrelated code
|
|
81
85
|
- **Keep commits focused**: One logical change per commit
|
|
82
86
|
- **Test before pushing**: Always run local checks first
|
|
83
|
-
- **
|
|
87
|
+
- **Handle informational comments**: If a comment (including from bots) is purely informational with no actionable request, acknowledge it internally and move on without code changes
|
|
@@ -1 +0,0 @@
|
|
|
1
|
-
import{ap as o,aq as n}from"./index-Cp1e-xYL.js";const t=(a,r)=>o.lang.round(n.parse(a)[r]);export{t as c};
|
|
@@ -1 +0,0 @@
|
|
|
1
|
-
import{s as a,c as s,a as e,C as t}from"./chunk-B4BG7PRW-EyCx-NZJ.js";import{_ as i}from"./index-Cp1e-xYL.js";import"./chunk-FMBD7UC4-D1FKmcwY.js";import"./chunk-55IACEB6-NeVyGiBw.js";import"./chunk-QN33PNHL-B-nhuti6.js";var p={parser:e,get db(){return new t},renderer:s,styles:a,init:i(r=>{r.class||(r.class={}),r.class.arrowMarkerAbsolute=r.arrowMarkerAbsolute},"init")};export{p as diagram};
|
|
@@ -1 +0,0 @@
|
|
|
1
|
-
import{s as a,c as s,a as e,C as t}from"./chunk-B4BG7PRW-EyCx-NZJ.js";import{_ as i}from"./index-Cp1e-xYL.js";import"./chunk-FMBD7UC4-D1FKmcwY.js";import"./chunk-55IACEB6-NeVyGiBw.js";import"./chunk-QN33PNHL-B-nhuti6.js";var p={parser:e,get db(){return new t},renderer:s,styles:a,init:i(r=>{r.class||(r.class={}),r.class.arrowMarkerAbsolute=r.arrowMarkerAbsolute},"init")};export{p as diagram};
|
|
@@ -1 +0,0 @@
|
|
|
1
|
-
import{b as r}from"./_baseUniq-DkCvE4O6.js";var e=4;function a(o){return r(o,e)}export{a as c};
|
|
@@ -1 +0,0 @@
|
|
|
1
|
-
import{s as t,b as r,a,S as s}from"./chunk-DI55MBZ5-PoOxaYU4.js";import{_ as i}from"./index-Cp1e-xYL.js";import"./chunk-55IACEB6-NeVyGiBw.js";import"./chunk-QN33PNHL-B-nhuti6.js";var l={parser:a,get db(){return new s(2)},renderer:r,styles:t,init:i(e=>{e.state||(e.state={}),e.state.arrowMarkerAbsolute=e.arrowMarkerAbsolute},"init")};export{l as diagram};
|