crewly 1.6.4 → 1.6.5

package/dist/backend/backend/src/services/agent/agent-registration.service.js

@@ -6,6 +6,7 @@ import { LoggerService } from '../core/logger.service.js';
 import { createSessionCommandHelper, getSessionBackendSync, createSessionBackend, getSessionStatePersistence, } from '../session/index.js';
 import { RuntimeServiceFactory } from './runtime-service.factory.js';
 import { registerInProcessRuntime, unregisterInProcessRuntime, } from './crewly-agent/in-process-runtime-registry.js';
+import { getCrewlyHomePath } from '../core/crewly-home.utils.js';
 import { CREWLY_CONSTANTS, ENV_CONSTANTS, AGENT_TIMEOUTS, ORCHESTRATOR_SESSION_NAME, ORCHESTRATOR_ROLE, RUNTIME_TYPES, SESSION_COMMAND_DELAYS, EVENT_DELIVERY_CONSTANTS, TERMINAL_PATTERNS, GEMINI_SHELL_MODE_CONSTANTS, GEMINI_STUCK_CONNECTIVITY_PATTERN, GEMINI_ERROR_STATE_CONSTANTS, } from '../../constants.js';
 import { WEB_CONSTANTS } from '../../../../config/constants.js';
 import { delay } from '../../utils/async.utils.js';
@@ -1319,7 +1320,10 @@ export class AgentRegistrationService {
             // NOW" before the supplementary memory context. Soft-fail with WARN
             // — TaskPool/Request hiccups must not block agent boot.
             try {
-                const activeWorkService = ActiveWorkBriefingService.getInstance();
+                // ActiveWorkBriefingService.getInstance() is async — see service
+                // JSDoc. Dynamic-import migration was the build-fix for PR #494's
+                // broken relative-path resolution under entry-script anchoring.
+                const activeWorkService = await ActiveWorkBriefingService.getInstance();
                 const activeWorkBriefing = await activeWorkService.generateActiveWorkBriefing(sessionName, role);
                 const activeWorkMd = activeWorkService.formatBriefingAsMarkdown(activeWorkBriefing);
                 if (activeWorkMd && activeWorkMd.length > 30) {
@@ -1510,7 +1514,19 @@ bash ${skillsPath}/core/register-self/execute.sh '{"role":"${role}","sessionName
 \`\`\`
 All it does is update a local status flag so the web UI shows you as online - nothing more.
 
-After checking in, just say "Ready for tasks" and wait for me to send you work.`;
+## Default Execution Loop
+
+When assigned a task, do not wait passively.
+
+Loop until done, blocked, or explicitly reassigned:
+1. Restate the expected outcome in one sentence.
+2. Identify the fastest safe path to produce a usable result.
+3. Execute immediately.
+4. Run cheapest meaningful validation.
+5. If validation fails, fix and retry.
+6. If blocked by missing goal/outcome/eval, escalate to your TL.
+7. If blocked by implementation detail, decide reasonably and continue.
+8. Report only when you have a result, blocker, or decision exceeding your authority.`;
         }
     }
     /**
@@ -1559,9 +1575,18 @@ After checking in, just say "Ready for tasks" and wait for me to send you work.`
     }
     /**
      * Build the unified init prompt path used by non-Claude-Code runtimes.
+     *
+     * Anchored on {@link getCrewlyHomePath} so test profiles, KR3 livewalk
+     * dry-runs, and ESTestNode acceptance harnesses that export
+     * `CREWLY_HOME=/tmp/...` write into the test profile, not the
+     * developer's real `~/.crewly/prompts`. WI-F (2026-05-07): the
+     * previous `path.join(os.homedir(), CREWLY_CONSTANTS.PATHS.CREWLY_HOME,
+     * ...)` form ignored the env var and let Mia's KR3 livewalk overwrite
+     * 27 occurrences of `~/.crewly/prompts/crewly-orc-init.md` and 26 of
+     * `crewly-auditor-init.md` in Steve's user-prod tree at 02:37Z.
      */
     getInitPromptFilePath(sessionName) {
-        return path.join(os.homedir(), CREWLY_CONSTANTS.PATHS.CREWLY_HOME, 'prompts', `${sessionName}-init.md`);
+        return path.join(getCrewlyHomePath(), 'prompts', `${sessionName}-init.md`);
     }
     /**
      * Wait for agent registration to complete