@ddtcorex/dsh-maestro-review 0.8.0 → 0.9.0

package/presets/maestro-reviewer.patch.yml

@@ -0,0 +1,55 @@
+# Agent preset Maestro Reviewer: declared as a bundle patch row so that every
+# profile composing @ddtcorex/dsh-maestro-review gets it. DSH 0.1.7 collects
+# preset definitions from loader rows naming @deepseek-ai/dsh-agent-preset; the
+# retired plural @deepseek-ai/dsh-agent-presets instead scanned
+# $DSH_HOME/.agent-presets, which is why the reviewer image used to COPY the
+# preset directories in. A profile that wants a different composition overrides
+# this row by bare id (no `insert:` wrapper) rather than inserting a second one.
+- insert:
+    - id: preset-dsh-maestro-reviewer
+      name: '@deepseek-ai/dsh-agent-preset'
+      config:
+        id: dsh-maestro-reviewer
+        name: Maestro Reviewer
+        description: >-
+          Automated AI Tech Lead that reviews a GitLab merge request's diff against project coding standards. Invoked programmatically by dsh-maestro-review's orchestrator — not meant for manual selection.
+        plugins:
+          - id: persona
+            name: '@deepseek-ai/dsh-persona'
+            config:
+              # The working directory varies per session, so it belongs in the suffix: the
+              # prefix is the model-stable part of the prompt that stays cacheable. The row
+              # must restate the suffix because an empty one shadows the deployment's away.
+              suffix: Your working directory is {{cwd}}.
+              prefix: |
+                You are Maestro Reviewer, an AI Tech Lead. Workflow:
+                1. If the review request names a profile, call maestro_load_review_profile with that profile before inspecting code — it fails closed if the required skill set is incomplete. For a diff-only review with no local checkout or Magento environment, skip this step: there is no profile to load, and the diff is all the context you need.
+                2. Call gitlab_list_own_review_threads to see which findings you already raised on this merge request.
+                3. Call gitlab_get_mr_diff to read the current diff.
+                4. For each existing thread whose file/line was touched by the new commits, decide whether it still applies, looks fixed, or needs a different comment — reply to it (status: "reply"). Never resolve a thread yourself.
+                5. For genuinely new issues with no existing thread, prepare a new inline comment (status: "new").
+                6. Call report_review_findings exactly once with your complete list of findings, then stop. Do not call gitlab_post_mr_comment, gitlab_post_inline_comment, or gitlab_reply_to_thread yourself — the orchestrator posts them from your report_review_findings call.
+
+          - id: skills-tool
+            name: '@ddtcorex/dsh-maestro-review/lib/skills-tool.js'
+
+          # Delegation tools: the providers live in the Profile
+          # (`subagent-codex` + `subagent-claude-code` rows), so a deployment without
+          # those bundles gets inert rows — `tool-subagent` registers no tool until its
+          # provider appears — rather than a failed mount. One-shot rows: a review
+          # delegates a bounded task and collects the result.
+          - id: tool-subagent-codex
+            name: '@deepseek-ai/dsh-tool-subagent'
+            config:
+              provider: codex
+              toolName: subagent_codex
+              backgroundMode: one-shot
+              maxDepth: provider-managed
+
+          - id: tool-subagent-claude-code
+            name: '@deepseek-ai/dsh-tool-subagent'
+            config:
+              provider: claude-code
+              toolName: subagent_claude_code
+              backgroundMode: one-shot
+              maxDepth: provider-managed

package/profiles/reviewer-ci/cordis.patch.yml

@@ -59,12 +59,17 @@
 
 # New rows this profile needs that dsh-base/dsh-maestro-review don't already provide.
 - insert:
-    # @deepseek-ai/dsh-agent-presets ships no dsh.bundle.patch of its own
-    # (confirmed — no "dsh" field in its package.json), so it never self-composes;
-    # every profile that wants it must insert this row itself, copied verbatim
-    # from deepseek-harness/packages/bundle/web-app/cordis.patch.yml.
-    - id: agent-presets
-      name: '@deepseek-ai/dsh-agent-presets'
+    # The agent-preset registry is the 0.1.7 provider: it collects preset
+    # definitions from loader rows, and this bundle now ships the reviewer and
+    # auditor declarations as patch rows of its own
+    # (./presets/maestro-{reviewer,auditor}.patch.yml), so no directory has to be
+    # copied anywhere. The retired plural @deepseek-ai/dsh-agent-presets is gone
+    # from this profile on purpose: it was only ever needed to read
+    # $DSH_HOME/.agent-presets. (dsh-base does not provide this row — the
+    # web-app bundle is what inserts it there — so a headless profile inserts it
+    # itself, copied from deepseek-harness/packages/bundle/web-app/cordis.patch.yml.)
+    - id: agent-preset-registry
+      name: '@deepseek-ai/dsh-agent-preset-registry'
       config:
         default: standard
 

package/profiles/reviewer-ci/package.json

@@ -11,9 +11,9 @@
     }
   },
   "dependencies": {
-    "@ddtcorex/dsh-maestro-review": "^0.7.5",
+    "@ddtcorex/dsh-maestro-review": "^0.8.1",
     "@ddtcorex/maestro-skills": "^2.13.0",
-    "@deepseek-ai/dsh": "0.1.5-rc.2",
-    "@deepseek-ai/dsh-agent-presets": "0.1.5-rc.2"
+    "@deepseek-ai/dsh": "0.1.7-rc.1",
+    "@deepseek-ai/dsh-agent-preset-registry": "0.1.7-rc.1"
   }
 }