@zumikua/dsh-junie-subagent 0.1.0 → 0.1.1

package/helpers.js

@@ -0,0 +1,94 @@
+/**
+ * Provider-side helpers for out-of-process subagent backends.
+ *
+ * These mirror the contracts of the `@deepseek-ai/dsh-subagent/out-of-process`
+ * vocabulary, but live here on purpose: a published bundle installs into a
+ * profile whose module resolution cannot see the Host's own packages, so the
+ * plugin must not import `@deepseek-ai/*` at runtime.
+ */
+
+import { existsSync } from 'node:fs'
+import { isAbsolute } from 'node:path'
+
+/** A child in another process cannot honour parent-enforced start features. */
+export const NO_START_CAPABILITIES = Object.freeze({
+  agentOptions: false,
+  agentPreset: false,
+  outputSchema: false,
+  depthLimit: false,
+})
+
+/**
+ * Reject a timing value the seam would refuse anyway, with the plugin's name in
+ * the message.
+ * @param prefix - the reporting plugin's diagnostic prefix.
+ * @param name - the configuration key being validated.
+ * @param value - the value to check.
+ */
+export function assertPositiveFinite(prefix, name, value) {
+  if (typeof value !== 'number' || !Number.isFinite(value) || value <= 0) {
+    throw new Error(`${prefix}: ${name} must be a positive finite number, received ${String(value)}`)
+  }
+}
+
+/**
+ * Reject a working directory the child process could not enter.
+ * @param prefix - the reporting plugin's diagnostic prefix.
+ * @param label - describes the source of the value, for the message.
+ * @param value - the candidate directory.
+ * @returns the same value, for chaining.
+ */
+export function assertUsableCwd(prefix, label, value) {
+  if (typeof value !== 'string' || value.length === 0 || !isAbsolute(value)) {
+    throw new Error(`${prefix}: ${label} must be an absolute path, received ${String(value)}`)
+  }
+  if (!existsSync(value)) throw new Error(`${prefix}: ${label} does not exist: ${value}`)
+  return value
+}
+
+/**
+ * Settle one run under the seam contract: `result` never rejects, and the
+ * abort listener the caller registered is removed on every path.
+ * @param parts - the attempt, the partial-output readers, and the abort wiring.
+ * @returns the settled result.
+ */
+export async function settleRunResult(parts) {
+  const { attempt, collectOutput, collectDiagnostic, cancelled, signal, onAbort } = parts
+  try {
+    return await attempt()
+  } catch (error) {
+    const diagnostic = collectDiagnostic()
+    return {
+      output: collectOutput(),
+      stopReason: cancelled() ? 'aborted' : 'error',
+      ...(diagnostic === undefined || diagnostic === '' ? {} : { diagnostic }),
+    }
+  } finally {
+    signal?.removeEventListener('abort', onAbort)
+  }
+}
+
+/**
+ * Publish the seam run handle for an out-of-process child. `dispose()` is
+ * idempotent: it removes the abort listener, requests cancellation without
+ * assuming the child cooperates, then awaits the backend teardown.
+ * @param parts - run identity, the settled result, cancellation, and teardown.
+ * @returns the external run handle.
+ */
+export function subprocessRunHandle(parts) {
+  const { id, result, signal, onAbort, requestCancel, teardown } = parts
+  let disposal
+  const dispose = () => {
+    disposal ??= (async () => {
+      signal?.removeEventListener('abort', onAbort)
+      try {
+        requestCancel()
+      } catch {
+        // Cancellation is best effort; teardown still owns the escalation.
+      }
+      await teardown()
+    })()
+    return disposal
+  }
+  return { id, result, dispose }
+}

package/index.js

@@ -20,8 +20,7 @@ import {
   assertUsableCwd,
   settleRunResult,
   subprocessRunHandle,
-} from '@deepseek-ai/dsh-subagent'
-import Schema from '@deepseek-ai/schemastery'
+} from './helpers.js'
 
 export const name = 'junie-subagent'
 
@@ -298,22 +297,11 @@ function configRouteHandler(ctx, cfg) {
 }
 
 /**
- * Row configuration, also the schema the settings service projects: `model`
- * and `effort` are the two values a person changes from the settings page.
+ * Note: this bundle deliberately exports no `Config` schema. A schemastery
+ * schema would need `@deepseek-ai/schemastery`, which is not resolvable from a
+ * profile-installed bundle; the row defaults are applied by `normalize()`
+ * instead, and the settings page is the supported way to change them.
  */
-export const Config = Schema.object({
-  providerName: Schema.string().default('junie'),
-  command: Schema.string().default(''),
-  args: Schema.array(Schema.string()).default([]),
-  model: Schema.string().default('mix.normal'),
-  effort: Schema.string().default('high'),
-  braveMode: Schema.string().default('brave-auto'),
-  permission: Schema.string().default('reject'),
-  timeoutMs: Schema.number().default(900000),
-  graceMs: Schema.number().default(3000),
-})
-
-
 
 /** ACP terminal reasons mapped onto the seam's stop-reason vocabulary. */
 const STOP_REASONS = new Map([

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zumikua/dsh-junie-subagent",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "publishConfig": {
     "access": "public"
   },
@@ -34,6 +34,7 @@
   },
   "files": [
     "index.js",
+    "helpers.js",
     "client.js",
     "cordis.patch.yml",
     "locale/*.json",
@@ -45,11 +46,6 @@
   "engines": {
     "dsh": ">=0.2.1-rc.2"
   },
-  "peerDependencies": {
-    "@deepseek-ai/cordis": "^4.0.4",
-    "@deepseek-ai/dsh-subagent": "0.2.1-rc.2",
-    "@deepseek-ai/schemastery": "^3.0.0"
-  },
   "dsh": {
     "bundle": {
       "patch": "./cordis.patch.yml"