@diff-review-system/drs 4.0.0-rc.4 → 4.1.0

package/.pi/workflows/gitlab-mr-visual-explain.yaml

@@ -0,0 +1,78 @@
+name: gitlab-mr-visual-explain
+description: Generate a visual HTML explainer artifact for a GitLab merge request
+inputs:
+  project: ''
+  mr: ''
+  outputPath: '.drs/visual-mr-explainer.html'
+  slides: 'false'
+output: visualExplainer
+nodes:
+  change:
+    action: change-source
+    with:
+      type: gitlab-mr
+      project: '{{inputs.project}}'
+      mr: '{{inputs.mr}}'
+    output: change
+
+  review-artifact:
+    action: artifact-exists
+    needs:
+      - change
+    with:
+      kind: review
+      source: change
+    output: reviewArtifactStatus
+
+  load-review-artifact:
+    action: load-artifact
+    needs:
+      - review-artifact
+    if: artifacts.reviewArtifactStatus.exists == true
+    with:
+      kind: review
+      source: change
+    output: reviewArtifact
+
+  visual-with-review:
+    agent: visual/pr-explainer
+    needs:
+      - change
+      - load-review-artifact
+    if: artifacts.reviewArtifactStatus.exists == true
+    input: |
+      Generate a visual MR explainer HTML artifact.
+
+      Output path: {{inputs.outputPath}}
+      Slide mode: {{inputs.slides}}
+
+      Use the supplied DRS change-source context. If important file diffs are omitted or summarized, use git_diff for those files before explaining them.
+
+      DRS change-source context:
+      {{artifacts.change}}
+
+      Current DRS review artifact path: {{artifacts.reviewArtifact.path}}
+
+      Use the read_artifact tool with this path to include confirmed DRS findings in a clearly labeled section. Do not assume the review JSON is in this prompt.
+    writes: '{{inputs.outputPath}}'
+    output: visualExplainer
+
+  visual:
+    agent: visual/pr-explainer
+    needs:
+      - review-artifact
+    if: artifacts.reviewArtifactStatus.exists == false
+    input: |
+      Generate a visual MR explainer HTML artifact.
+
+      Output path: {{inputs.outputPath}}
+      Slide mode: {{inputs.slides}}
+
+      Use the supplied DRS change-source context. If important file diffs are omitted or summarized, use git_diff for those files before explaining them.
+
+      DRS change-source context:
+      {{artifacts.change}}
+
+      No current DRS review artifact exists for this MR. Label the artifact as an MR explainer with no DRS findings included.
+    writes: '{{inputs.outputPath}}'
+    output: visualExplainer

package/.pi/workflows/local-fix-review-issues.yaml

@@ -1,27 +1,62 @@
 name: local-fix-review-issues
-description: Fix actionable issues from a saved DRS review result
+description: Fix actionable issues from a saved local DRS review artifact with verification loop
+metadata:
+  kind: review
+  tags: [review, fix, local]
+  review:
+    source: local
+    diff: true
+    issues: true
 inputs:
-  review: ""
+  review:
+    type: string
+    description: Saved review artifact id; empty loads the latest artifact for the current scope
+  staged:
+    type: boolean
+    default: false
+  fixSeverity:
+    type: enum
+    values: [critical, high, medium, low]
+    default: high
+  fixMaxIterations:
+    type: number
+    default: 3
+output: reviewArtifact
 nodes:
   change:
     action: change-source
     with:
       type: local
-      staged: false
+      staged: '{{inputs.staged}}'
     output: change
 
+  load-review-artifact:
+    action: load-artifact
+    needs:
+      - change
+    with:
+      kind: review
+      source: change
+      id: '{{inputs.review}}'
+    output: reviewArtifact
+
   fix-issues:
     agent: task/review-issue-fixer
     needs:
-      - change
+      - load-review-artifact
     input: |
-      Fix actionable issues from this DRS review result.
+      Fix actionable issues from this saved local DRS review artifact.
+
+      Only fix findings at or above severity {{inputs.fixSeverity}} unless a lower-severity issue is directly required by such a fix.
+
+      Review artifact path: {{artifacts.reviewArtifact.path}}
 
-      Review result:
-      {{inputs.review}}
+      Use the read_artifact tool with this path to get the finding manifest, then pull full details for specific findings by id. Do not assume the review JSON is in this prompt — call the tool.
 
-      Current local change source:
-      {{artifacts.change}}
+      Current local change source files (read these as needed):
+      {{artifacts.change.files}}
+
+      Use the drs_check tool after making changes to run configured validation checks before returning.
     output: fixes
 
   final-change:
@@ -30,13 +65,54 @@ nodes:
       - fix-issues
     with:
       type: local
-      staged: false
+      staged: '{{inputs.staged}}'
     output: finalChange
 
-  review:
+  verification-change:
+    action: change-source
+    needs:
+      - final-change
+    with:
+      type: fix-verification
+      source: change
+      fixChange: finalChange
+    output: verificationChange
+
+  re-review:
     action: review
     needs:
+      - verification-change
+    with:
+      source: verificationChange
+      reviewArtifact: reviewArtifact
+      severity: '{{inputs.fixSeverity}}'
+    output: reReview
+
+  verify-fix:
+    action: verify-fix
+    needs:
+      - re-review
+      - load-review-artifact
       - final-change
     with:
-      source: finalChange
-    output: review
+      artifact: reviewArtifact
+      review: reReview
+      fixChange: finalChange
+      severity: '{{inputs.fixSeverity}}'
+      minIssues: '1'
+    output: reviewArtifact
+
+  fix-loop:
+    control: loop
+    needs:
+      - verify-fix
+    if: artifacts.verify-fix.shouldContinue == true
+    target: fix-issues
+    exit: done
+    maxIterations: '{{inputs.fixMaxIterations}}'
+    onMaxIterations: exit
+
+  done:
+    control: end
+    needs:
+      - fix-loop

package/.pi/workflows/local-review.yaml

@@ -1,11 +1,24 @@
 name: local-review
-description: Review local unstaged git diff
+description: Review local git diff
+metadata:
+  kind: review
+  tags: [review, local]
+  review:
+    source: local
+    diff: true
+    issues: true
+inputs:
+  staged:
+    type: boolean
+    default: false
+    description: Review staged changes instead of unstaged changes
+output: review
 nodes:
   change:
     action: change-source
     with:
       type: local
-      staged: false
+      staged: '{{inputs.staged}}'
     output: change
 
   review:

package/.pi/workflows/local-update-agents-md.yaml

@@ -1,7 +1,7 @@
 name: local-update-agents-md
 description: Update repository agent guidance from local changes
 inputs:
-  path: ""
+  path: ''
 nodes:
   change:
     action: change-source

package/.pi/workflows/local-visual-explain.yaml

@@ -0,0 +1,76 @@
+name: local-visual-explain
+description: Generate a visual HTML explainer artifact for a local git diff
+inputs:
+  staged: 'false'
+  outputPath: '.drs/visual-local-explainer.html'
+  slides: 'false'
+output: visualExplainer
+nodes:
+  change:
+    action: change-source
+    with:
+      type: local
+      staged: '{{inputs.staged}}'
+    output: change
+
+  review-artifact:
+    action: artifact-exists
+    needs:
+      - change
+    with:
+      kind: review
+      source: change
+    output: reviewArtifactStatus
+
+  load-review-artifact:
+    action: load-artifact
+    needs:
+      - review-artifact
+    if: artifacts.reviewArtifactStatus.exists == true
+    with:
+      kind: review
+      source: change
+    output: reviewArtifact
+
+  visual-with-review:
+    agent: visual/pr-explainer
+    needs:
+      - change
+      - load-review-artifact
+    if: artifacts.reviewArtifactStatus.exists == true
+    input: |
+      Generate a visual local-diff explainer HTML artifact.
+
+      Output path: {{inputs.outputPath}}
+      Slide mode: {{inputs.slides}}
+
+      Use the supplied DRS change-source context. If important file diffs are omitted or summarized, use git_diff for those files before explaining them.
+
+      DRS change-source context:
+      {{artifacts.change}}
+
+      Current DRS review artifact path: {{artifacts.reviewArtifact.path}}
+
+      Use the read_artifact tool with this path to include confirmed DRS findings in a clearly labeled section. Do not assume the review JSON is in this prompt.
+    writes: '{{inputs.outputPath}}'
+    output: visualExplainer
+
+  visual:
+    agent: visual/pr-explainer
+    needs:
+      - review-artifact
+    if: artifacts.reviewArtifactStatus.exists == false
+    input: |
+      Generate a visual local-diff explainer HTML artifact.
+
+      Output path: {{inputs.outputPath}}
+      Slide mode: {{inputs.slides}}
+
+      Use the supplied DRS change-source context. If important file diffs are omitted or summarized, use git_diff for those files before explaining them.
+
+      DRS change-source context:
+      {{artifacts.change}}
+
+      No current DRS review artifact exists for this diff. Label the artifact as a diff explainer with no DRS findings included.
+    writes: '{{inputs.outputPath}}'
+    output: visualExplainer

package/.pi/workflows/release-changelog-finalize.yaml

@@ -0,0 +1,47 @@
+name: release-changelog-finalize
+description: Finalize CHANGELOG.md for a release from an explicit git range
+inputs:
+  from:
+    type: string
+    description: Base ref or previous release tag
+    required: true
+  to:
+    type: string
+    default: HEAD
+    description: Release candidate ref, final release commit, or HEAD
+  version:
+    type: string
+    description: Release version without leading v, for example 4.0.0
+    required: true
+  date:
+    type: string
+    description: Release date in YYYY-MM-DD format
+    required: true
+output: changelog
+nodes:
+  release-change:
+    action: change-source
+    with:
+      type: git-range
+      from: '{{inputs.from}}'
+      to: '{{inputs.to}}'
+    output: change
+
+  finalize-changelog:
+    agent: task/changelog-updater
+    needs:
+      - release-change
+    input: |
+      Finalize CHANGELOG.md for release {{inputs.version}} dated {{inputs.date}}.
+
+      Use this git range change source as the authoritative release diff:
+      {{artifacts.change}}
+
+      Requirements:
+      - Ensure the file starts with `# Changelog` followed by the standard introduction.
+      - Move release-worthy `## Unreleased` entries into `## {{inputs.version}} - {{inputs.date}}`.
+      - Create `## {{inputs.version}} - {{inputs.date}}` directly above the previous released version if it does not exist.
+      - Keep `## Unreleased` above the release section only if there are remaining unreleased entries; otherwise remove empty duplicate/unneeded headings.
+      - Preserve existing older release sections.
+      - Do not create git tags, publish packages, or edit files other than CHANGELOG.md.
+    output: changelog

package/.pi/workflows/tag-changelog-update.yaml

@@ -1,15 +1,15 @@
 name: tag-changelog-update
 description: Update CHANGELOG.md from changes between two git refs or tags
 inputs:
-  from: ""
-  to: ""
+  from: ''
+  to: ''
 nodes:
   release-change:
     action: change-source
     with:
       type: git-range
-      from: "{{inputs.from}}"
-      to: "{{inputs.to}}"
+      from: '{{inputs.from}}'
+      to: '{{inputs.to}}'
     output: change
 
   update-changelog:

package/.pi/workflows/temporal-control-smoke.yaml

@@ -0,0 +1,63 @@
+name: temporal-control-smoke
+description: Smoke test Temporal control-flow execution with safe local write actions
+inputs:
+  mode:
+    type: enum
+    values:
+      - loop
+      - pass
+      - end
+    default: loop
+    description: 'Control route to exercise: loop, passThrough, or end'
+output: final
+nodes:
+  start:
+    action: write
+    input: '{{inputs.mode}}'
+    writes: .drs/artifacts/temporal-smoke/start.txt
+    output: mode
+
+  route:
+    control: switch
+    needs:
+      - start
+    value: '{{artifacts.mode}}'
+    cases:
+      loop: loop-body
+      end: stop
+    default: pass
+
+  loop-body:
+    action: write
+    needs:
+      - route
+    input: 'loop body for {{artifacts.mode}}'
+    writes: .drs/artifacts/temporal-smoke/loop-body.txt
+    output: loopBody
+
+  again:
+    control: loop
+    needs:
+      - loop-body
+    if: 'true'
+    target: loop-body
+    exit: done
+    maxIterations: 2
+    onMaxIterations: exit
+
+  pass:
+    control: passThrough
+    needs:
+      - route
+    target: done
+
+  stop:
+    control: end
+    needs:
+      - route
+
+  done:
+    action: write
+    input: 'temporal control smoke complete via {{inputs.mode}}'
+    writes: .drs/artifacts/temporal-smoke/done.txt
+    output: final