@sweny-ai/core 0.1.177 → 0.1.178

package/dist/workflows/implement.yml

@@ -23,7 +23,7 @@ nodes:
          identifier wrapped in brackets — e.g. query
          `[OFF-1234] in:title type:pr state:open`. If the response contains
          one or more PRs whose title starts with `[<issueIdentifier>]`,
-         STOP analysis. Output `existing_pr_url`
+         STOP analysis. Output `has_open_pr: true`, `existing_pr_url`
          (the URL of the open PR) and a short `issue_summary` noting the
          skip; leave `fix_plan` set to a one-line "Skip: open PR
          <url> already exists". The workflow will route to `notify` and
@@ -37,7 +37,12 @@ nodes:
       4. Identify the exact files and lines that need to change.
       5. Plan the fix approach.
 
-      Output a clear analysis of what needs to change and why.
+      Output a clear analysis of what needs to change and why, plus three
+      routing fields: `has_open_pr` (true only when step 2 found an open PR),
+      `risk_level` (low, medium or high), and `plan_is_clear` (true only when
+      the plan names the exact files and changes). sweny implements the fix
+      only when there is no open PR, the risk is low or medium, and the plan
+      is clear; otherwise it skips.
     skills:
       - github
       - linear
@@ -68,9 +73,23 @@ nodes:
             and the analyze node is short-circuiting. Triggers the
             `analyze → notify` edge.
           pattern: "^https?://"
+        has_open_pr:
+          type: boolean
+          description: |
+            true when step 2 found an open PR for this issue (and
+            existing_pr_url is set), else false. Routes analyze to notify.
+        plan_is_clear:
+          type: boolean
+          description: |
+            true when fix_plan names the exact files and changes, so the fix
+            can be implemented without guessing; false when the fix is
+            unclear or needs a decision first.
       required:
         - issue_summary
         - fix_plan
+        - has_open_pr
+        - risk_level
+        - plan_is_clear
   implement:
     name: Implement Fix
     instruction: |-
@@ -182,15 +201,28 @@ nodes:
         max: 1
         number: { input: issueIdentifier }
 edges:
+  # Deterministic routing (#357), equivalent to the old natural-language
+  # conditions over declared fields: an open PR (has_open_pr, or
+  # existing_pr_url set, as the old condition read it) goes to notify;
+  # otherwise risk low or medium with a clear plan goes to implement;
+  # anything else (high risk or an unclear plan) takes the default edge to
+  # skip. The two expressions are exclusive (the implement one requires no
+  # open PR by either signal). A routing field still missing or invalid after
+  # sweny's repair request falls through to the agent, which reads the
+  # descriptions.
   - from: analyze
     to: notify
-    when: analyze.existing_pr_url is set (an open PR for this issue already exists; skip the implement and create_pr nodes)
+    when:
+      expr: "analyze.has_open_pr == true || exists analyze.existing_pr_url"
+    description: an open PR for this issue already exists (analyze.existing_pr_url is set); skip the implement and create_pr nodes
   - from: analyze
     to: implement
-    when: Fix risk level is low or medium and a clear plan exists
+    when:
+      expr: "analyze.has_open_pr == false && !(exists analyze.existing_pr_url) && analyze.risk_level in ['low', 'medium'] && analyze.plan_is_clear == true"
+    description: no open PR exists, the fix risk level is low or medium, and a clear plan exists
   - from: analyze
     to: skip
-    when: Fix is too complex, risky, or unclear
+    description: the fix is too complex, risky, or unclear
   - from: implement
     to: create_pr
   - from: skip

package/dist/workflows/triage.yml

@@ -109,6 +109,9 @@ nodes:
 
       - `novel_count`: how many findings are novel (not duplicates).
 
+      - `fixable_count`: how many NOVEL findings have `fix_complexity` simple or moderate AND a non-empty
+      `fix_approach`. 0 when none do. This decides whether sweny attempts a fix after filing the issue.
+
       - `highest_severity`: the highest severity across ALL findings.
 
       - `recommendation`: what should happen next.
@@ -190,6 +193,19 @@ nodes:
         novel_count:
           type: number
           description: Count of novel (non-duplicate) findings
+        fixable_count:
+          type: integer
+          minimum: 0
+          description: >-
+            Count of novel findings whose fix_complexity is simple or moderate AND
+            whose fix_approach is a non-empty plan. Routes create_issue to implement.
+          # sweny recounts this from `findings` and treats a disagreeing count as invalid (#357).
+          x-sweny-count-of:
+            array: findings
+            where:
+              is_duplicate: [false]
+              fix_complexity: [simple, moderate]
+            non_empty: [fix_approach]
         highest_severity:
           type: string
           enum:
@@ -202,6 +218,7 @@ nodes:
       required:
         - findings
         - novel_count
+        - fixable_count
         - highest_severity
         - recommendation
   create_issue:
@@ -796,33 +813,50 @@ edges:
   # Deterministic routing (#461): sweny evaluates these expressions over the
   # declared, required outputs of investigate, with no model call. The two
   # are complements over the declared enum, so exactly one is true.
+  #
+  # Every expression edge carries a `description`: the same condition in
+  # natural language. When a field an expression needs is still missing or
+  # invalid after sweny's one repair request, the route falls through to the
+  # agent, which decides on these descriptions (#357).
   - from: investigate
     to: create_issue
     when:
       expr: "investigate.novel_count > 0 && investigate.highest_severity in ['medium', 'high', 'critical']"
+    description: at least one novel finding, and the highest severity is medium or higher
   - from: investigate
     to: skip
     when:
       expr: "investigate.novel_count == 0 || investigate.highest_severity == 'low'"
+    description: no novel findings, or the highest severity is low
+  # Deterministic too (#357): fixable_count counts exactly the novel findings
+  # the old natural-language condition asked about (fix_complexity simple or
+  # moderate AND a fix_approach), so `> 0` is that condition and the default
+  # edge is its complement. sweny cross-checks the count against the
+  # findings (x-sweny-count-of): a count that disagrees with the findings is
+  # invalid, which reads as missing and falls through to the agent.
   - from: create_issue
     to: implement
-    when: at least one novel finding has fix_complexity simple or moderate AND fix_approach is provided
+    when:
+      expr: "investigate.fixable_count > 0"
+    description: at least one novel finding has fix_complexity simple or moderate AND fix_approach is provided
   - from: create_issue
     to: notify
-    when: all novel findings have fix_complexity complex, OR no clear fix_approach
+    description: all novel findings have fix_complexity complex, OR no clear fix_approach
   - from: skip
     to: notify
-  # test_status is required and the node's eval only lets pass, no-framework
-  # and skipped through (fail and not-run fail the node, which halts), so
-  # exactly one of these is true. skipped: Step 0b found an open PR for the
-  # issue, or Step 0 found a staged preview.
+  # The node's eval only lets test_status pass, no-framework and skipped
+  # through (fail and not-run fail the node, which halts), so exactly one of
+  # these is true. skipped: Step 0b found an open PR for the issue, or Step 0
+  # found a staged preview.
   - from: implement
     to: notify
     when:
       expr: "implement.test_status == 'skipped'"
+    description: the implement step was skipped (an open PR already exists, or a staged preview was found)
   - from: implement
     to: create_pr
     when:
       expr: "implement.test_status in ['pass', 'no-framework']"
+    description: the fix is committed and its tests passed (or the repo has no test framework)
   - from: create_pr
     to: notify

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sweny-ai/core",
-  "version": "0.1.177",
+  "version": "0.1.178",
   "type": "module",
   "bin": {
     "sweny": "./dist/cli/main.js"