@goodandready/dsh-goal 0.2.13 → 0.2.14

package/lib/index.js

@@ -13,7 +13,7 @@ import { GoalEngine, GoalState } from './goal-engine.js';
 import { sessionIdOf } from './goal-engine-constants.js';
 
 export const name = '@goodandready/dsh-goal';
-export const inject = ['webServer', 'settings', 'goals?'];
+export const inject = ['webServer', 'settings'];
 
 const NS = 'dsh-goal';
 
@@ -39,6 +39,24 @@ export const Config = z.object({
   storagePath: z.string().default('').description('Custom filesystem path for persistent state storage'),
 });
 
+/**
+ * Safely access a service from Cordis context without throwing if un-injected.
+ * @param {any} ctx
+ * @param {string} name
+ * @returns {any}
+ */
+export function safeService(ctx, name) {
+  if (!ctx) return null;
+  try {
+    if (ctx.reflect && typeof ctx.reflect.get === "function") {
+      return ctx.reflect.get(name, false) || null;
+    }
+    return ctx[name] || null;
+  } catch {
+    return null;
+  }
+}
+
 export function apply(ctx, config = {}) {
   let settingsScope = null;
   const sseClients = new Map();
@@ -90,8 +108,9 @@ export function apply(ctx, config = {}) {
 
   // Attempt migration of legacy state if present
   try {
-    const firstAgent = ctx.agents?.get?.(ctx.agents?.keys?.()?.next()?.value);
-    addonStore.migrateLegacyState(storagePath, ctx.goals, firstAgent);
+    const agentsSvc = safeService(ctx, "agents");
+    const firstAgent = agentsSvc?.get?.(agentsSvc?.keys?.()?.next()?.value);
+    addonStore.migrateLegacyState(storagePath, safeService(ctx, "goals"), firstAgent);
   } catch (migErr) {
     logger.warn?.('[dsh-goal] Legacy migration check failed:', migErr.message);
   }
@@ -99,13 +118,13 @@ export function apply(ctx, config = {}) {
   // Register addon tools: goal_milestones and goal_checklist
   registerTools(ctx, {
     milestoneManager,
-    goalsService: ctx.goals,
+    goalsService: safeService(ctx, "goals"),
     logger,
   });
 
   // Support fallback engine for tests or non-core environments
   let fallbackEngine = null;
-  if (!ctx.goals) {
+  if (!safeService(ctx, "goals")) {
     fallbackEngine = new GoalEngine({
       storagePath,
       defaultMaxIterations: currentSettings.maxIterations,
@@ -158,7 +177,7 @@ export function apply(ctx, config = {}) {
     gitSnapshotManager,
     tokenBudgetGuard,
     addonStore,
-    goalsService: ctx.goals,
+    goalsService: safeService(ctx, "goals"),
     getConfig: () => currentSettings,
     sseClients,
     logger,
@@ -202,9 +221,10 @@ export function apply(ctx, config = {}) {
 
     // Inject milestone prompts and check budget during agent turns
     ctx.on('agent/pre-step', async ({ agent, messages, step, signal }, next) => {
-      if (ctx.goals && typeof ctx.goals.get === 'function' && agent) {
+      const goalsSvc = safeService(ctx, "goals");
+      if (goalsSvc && typeof goalsSvc.get === "function" && agent) {
         try {
-          const currentGoal = ctx.goals.get(agent);
+          const currentGoal = goalsSvc.get(agent);
           if (currentGoal && currentGoal.phase === 'active') {
             const goalId = currentGoal.id;
 
@@ -213,8 +233,8 @@ export function apply(ctx, config = {}) {
             const budgetCheck = tokenBudgetGuard.checkBudget({
               agent,
               goal: currentGoal,
-              tokenMeter: ctx.tokenMeter,
-              goalsService: ctx.goals,
+              tokenMeter: safeService(ctx, "tokenMeter"),
+              goalsService: goalsSvc,
               milestones,
             });
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@goodandready/dsh-goal",
-  "version": "0.2.13",
+  "version": "0.2.14",
   "description": "Core Goal Addon (Token Budget, Milestones, Git Snapshots & Retrospective Reports) for DeepSeek Harness",
   "type": "module",
   "main": "./lib/index.js",
@@ -63,5 +63,10 @@
   },
   "dependencies": {
     "@deepseek-ai/schemastery": "^3.18.1"
+  },
+  "peerDependenciesMeta": {
+    "@deepseek-ai/dsh-goal": {
+      "optional": true
+    }
   }
 }