@wowyuarm/dsh-agent-team 0.1.13 → 0.1.14-alpha.2

package/packages/agent-team/lib/index.js

@@ -52,13 +52,12 @@ import { Remote, TypertRemoteService } from '@deepseek-ai/dsh-typert-protocol';
 import { ATTACHMENT_MAX_BYTES, attachmentPayloadPath, attachmentsRoot, copyPathAttachment, newAttachmentId, readAttachment, sanitizeMediaType, sweepAttachmentCache, validatePathAttachment, writeAttachment } from "./attachments.js";
 import { PressurePolicyCoordinator } from "./pressure-policy.js";
 import { ContextManagementCoordinator } from "./context-management.js";
-import { AGENT_TEAM_PLUGIN_ID, createHandoffMessage } from "./context-source.js";
+import { AGENT_TEAM_PLUGIN_ID, createHandoffMessage, isAgentTeamSource, isAgentTeamSourceKind } from "./context-source.js";
 import { carriedInputOf, checkpointByRef, checkpointRefFor, contextProjectionFold, foldContextProjection, isReminderNoticeSummary, timelineCandidates } from "./context-projection.js";
 import { advanceOwnedSessionEventCursor } from "./session-event-cursor.js";
 import { AGENT_TEAM_HUMAN_MEMBER_ID, AgentTeamLedger, agentTeamHumanActor } from "./ledger.js";
 import { AGENT_TEAM_TOOL_NAMES, deepCopyCapabilities, memberMemoryDirectoryName, MemberRuntime } from "./member-runtime.js";
 import { classifyRecoverableError, RecoveryCoordinator, RECOVERY_MAX_CONSECUTIVE_ERRORS } from "./recovery.js";
-import { SessionRemediation, handoffAlreadyInLog } from "./session-remediation.js";
 import { StoredSessionReadError, StoredSessionReader, sessionFailureOf } from "./stored-session-reader.js";
 import { agentTeamDomainSpec } from "./spec.js";
 import { formatTeamTimestamp } from "./time-format.js";
@@ -351,12 +350,6 @@ let AgentTeam = (() => {
          */
         presenceEpoch = 0;
         changeWaiters = new Set();
-        /**
-         * The startup-opened remediation instance, held for the restart heal: the
-         * completion-cache domain may only be opened once per plugin lifecycle, so
-         * the restart path reuses this instance instead of opening its own.
-         */
-        remediation;
         constructor(ctx) {
             super(ctx, 'agentTeam');
             this.pressurePolicy = new PressurePolicyCoordinator({
@@ -445,7 +438,6 @@ let AgentTeam = (() => {
             const domain = await this.ctx.storageDomain.open(agentTeamDomainSpec);
             this.ctx.effect(() => async () => {
                 this.accepting = false;
-                this.remediation = undefined;
                 this.recovery.dispose();
                 this.contextManagement.dispose();
                 this.pressurePolicy.dispose();
@@ -468,18 +460,6 @@ let AgentTeam = (() => {
             if (initialization.committed)
                 this.emitCommitted(initialization.value);
             this.startAttachmentGc(ledger);
-            // Legacy-artifact remediation runs before any Member activation: no write
-            // lease exists yet, so publishing sibling generations for refused Session
-            // logs cannot race a live writer. Remediation failure never blocks
-            // startup — the next start retries exactly what the cache does not cover.
-            try {
-                const remediation = new SessionRemediation(this.ctx, this.ctx.sessionPersistence, await SessionRemediation.open(this.ctx));
-                this.remediation = remediation;
-                await remediation.remediateEnabledMembers(ledger.listMembers());
-            }
-            catch (error) {
-                this.ctx.logger.warn(`agent-team: legacy Session remediation did not run to completion (it will retry on the next start): ${error instanceof Error ? error.message : String(error)}`);
-            }
             // One metadata listing serves every Member restore; per-member list calls
             // would repeat the same I/O linearly during startup.
             const persistedSessions = new Set((await this.persistedSessionHeaders()).map(snapshot => snapshot.header.id));
@@ -718,23 +698,10 @@ let AgentTeam = (() => {
                 if (member.state !== 'enabled')
                     throw new Error(`Agent Member '${member.handle}' is ${member.state}; only enabled Members can be restarted`);
                 this.ctx.logger.info(`agent-team: restarting member '${member.handle}' after a failed activation`);
-                // A deterministic session refusal may be repairable in place: run the
-                // same bounded startup remediation for this one Member before retrying
-                // activation, so the restart heals instead of replaying the failure.
-                const activation = this.memberFailures.get(request.memberId)?.activation;
-                if (activation !== undefined && activation.class === 'session-refused' && this.remediation !== undefined) {
-                    const outcome = await this.remediation.remediateMember(member);
-                    if (outcome.repaired > 0) {
-                        this.ctx.logger.info(`agent-team: repaired ${outcome.repaired} refused Session artifact(s) for member '${member.handle}'; retrying activation`);
-                    }
-                    else if (outcome.completed) {
-                        // The walk finished and nothing was provably this plugin's to fix:
-                        // a retry would fail identically. Mark the refusal non-remediable
-                        // so the surface stops offering restart and says why.
-                        this.markRefusalNonRemediable(request.memberId);
-                        return Object.freeze({ status: this.memberStatus(member) });
-                    }
-                }
+                // There is no write-side repair pass anymore: dsh 0.1.7 converts the
+                // released V3 history at read time, and this bundle no longer authors
+                // the old wrapper shape, so a refusal stays a deterministic failure the
+                // retry reports again rather than something the restart heals.
                 await this.reactivateMember(request.memberId);
                 return Object.freeze({ status: this.memberStatus(member) });
             }
@@ -946,7 +913,7 @@ let AgentTeam = (() => {
             const body = notifications.length === 0 ? text : `${text}\n\n${this.notificationText(notifications, member.memberId)}`;
             const hint = createUserMessage({
                 content: [{ type: 'text', text: body }],
-                source: { kind: 'plugin', plugin: AGENT_TEAM_PLUGIN_ID, form: 'notice', summary: RECOVERY_NOTICE_SUMMARY },
+                source: { kind: AGENT_TEAM_PLUGIN_ID, form: 'notice', summary: RECOVERY_NOTICE_SUMMARY },
             });
             for (const pending of [...handle.agent.inbox.nextStep, ...handle.agent.inbox.nextTurn]) {
                 if (this.isInboxNotice(pending))
@@ -1317,7 +1284,7 @@ let AgentTeam = (() => {
             try {
                 const message = createUserMessage({
                     content: [{ type: 'text', text: this.dmRelayText(agent, recipient, request.body.trim(), result.value.receipt.occurredAt, result.value.receipt.operationId, request.workspaceId) }],
-                    source: { kind: 'plugin', plugin: AGENT_TEAM_PLUGIN_ID, form: 'relay' },
+                    source: { kind: AGENT_TEAM_PLUGIN_ID, form: 'relay' },
                 });
                 // An idle recipient gets one ordinary turn; a busy one is steered into
                 // its current turn — the same wake split subagent continuations use.
@@ -1866,11 +1833,11 @@ let AgentTeam = (() => {
                     openAttributions.set(event.data.callId, { name: event.data.name, arguments: event.data.arguments });
                 }
                 else if (event.type === 'tool/result') {
-                    const block = event.data.message.content?.[0];
-                    if (block !== undefined && block.toolCallId !== undefined) {
-                        const recorded = openAttributions.get(block.toolCallId);
-                        if (recorded !== undefined && block.isError !== true) {
-                            openAttributions.delete(block.toolCallId);
+                    const result = event.data.message;
+                    if (result.isError !== true) {
+                        const recorded = openAttributions.get(result.toolCallId);
+                        if (recorded !== undefined) {
+                            openAttributions.delete(result.toolCallId);
                             try {
                                 const args = JSON.parse(recorded.arguments);
                                 // A claim mutation resolves its Task overlay through the ledger;
@@ -1900,7 +1867,11 @@ let AgentTeam = (() => {
                 }
                 else if (event.type === 'user/message') {
                     const source = event.data.source;
-                    if (source?.kind !== 'plugin')
+                    // Only this plugin's own delivered notices attribute Threads, by exact
+                    // kind identity — the shape written now and the read-time conversion's
+                    // rename of the released V3 history. A `plugin:` prefix test would
+                    // claim third-party producers' rows as Team facts.
+                    if (source === undefined || !isAgentTeamSourceKind(source.kind))
                         continue;
                     // Reminder notices never enter attribution — a recovery instruction
                     // (or a historical progress-nudge notice, kept decodable in session
@@ -2324,12 +2295,11 @@ let AgentTeam = (() => {
                     // delivered activates the new Session with no handoff in its own
                     // log — never treat that as an ordinary blank Member Session. The
                     // operation's recorded previous Session (the lineage parent) still
-                    // holds the intent; rebuild the handoff from it. "No handoff in its
-                    // own log" is judged over both shapes: a generation rescued from the
-                    // retired custom kinds carries its handoff as a source the projection
-                    // does not classify, and rebuilding on top of it would inject the same
-                    // handoff twice.
-                    if (!handoffAlreadyInLog(state.boundaries, created.agent.session.ownEvents())) {
+                    // holds the intent; rebuild the handoff from it. Presence is judged
+                    // by the projection boundary alone: every handoff this Host ever
+                    // published classifies under the context source's recognizer,
+                    // including the history the read-time conversion renamed.
+                    if (!state.boundaries.some(boundary => boundary.source === 'handoff')) {
                         await this.reconstructMissingHandoff(member, created.agent);
                     }
                     // Carried input redelivery binds to the committed transition target —
@@ -2503,14 +2473,6 @@ let AgentTeam = (() => {
             }
             return { class: 'activation', detail: error instanceof Error ? error.message : String(error) };
         }
-        /** Mark a session-refused activation diagnostic as proven non-remediable. */
-        markRefusalNonRemediable(memberId) {
-            const failures = this.memberFailures.get(memberId);
-            const activation = failures?.activation;
-            if (activation === undefined || activation.class !== 'session-refused')
-                return;
-            failures.activation = Object.freeze({ ...activation, remediable: false });
-        }
         clearMemberFailure(memberId, slot) {
             const failures = this.memberFailures.get(memberId);
             if (failures === undefined || failures[slot] === undefined)
@@ -2551,7 +2513,7 @@ let AgentTeam = (() => {
                     const path = this.ctx.workspaceRegistry.get(operation.data.workspaceId)?.path;
                     const text = `Team participation changed: you have ${operation.kind === 'team/member-workspace-joined' ? 'joined' : 'left'} Workspace ${operation.data.workspaceId}${path === undefined ? ' (path unavailable)' : ` (${JSON.stringify(path)})`}.\nCurrent Workspace ids: ${ledger.workspacesOf(operation.data.memberId).join(', ')}. This replaces earlier participation information. Your Session and cwd have not moved.`;
                     const notice = createUserMessage({ content: [{ type: 'text', text }],
-                        source: { kind: 'plugin', plugin: AGENT_TEAM_PLUGIN_ID, form: 'notice', summary: 'Team Workspace participation changed' } });
+                        source: { kind: AGENT_TEAM_PLUGIN_ID, form: 'notice', summary: 'Team Workspace participation changed' } });
                     try {
                         if (agent.status === 'idle' || agent.inbox.nextTurn.some(message => message.source.kind === 'user'))
                             agent.followup(notice);
@@ -2648,7 +2610,7 @@ let AgentTeam = (() => {
                 agent.inbox.remove(existingInboxHint.id);
             const hint = createUserMessage({
                 content: [{ type: 'text', text: this.notificationText(notifications, member.memberId) }],
-                source: { kind: 'plugin', plugin: AGENT_TEAM_PLUGIN_ID, form: 'notice', summary: INBOX_NOTICE_SUMMARY },
+                source: { kind: AGENT_TEAM_PLUGIN_ID, form: 'notice', summary: INBOX_NOTICE_SUMMARY },
             });
             this.notifiedInbox.set(member.memberId, signature);
             try {
@@ -2669,12 +2631,12 @@ let AgentTeam = (() => {
         }
         isInboxNotice(message) {
             const source = message.source;
-            return source.kind === 'plugin' && source.plugin === AGENT_TEAM_PLUGIN_ID
+            return isAgentTeamSource(source)
                 && source.form === 'notice' && source.summary === INBOX_NOTICE_SUMMARY;
         }
         isRecoveryNotice(message) {
             const source = message.source;
-            return source.kind === 'plugin' && source.plugin === AGENT_TEAM_PLUGIN_ID
+            return isAgentTeamSource(source)
                 && source.form === 'notice' && source.summary === RECOVERY_NOTICE_SUMMARY;
         }
         notificationText(notifications, readerId) {

package/packages/agent-team/lib/member-context.js

@@ -2,6 +2,7 @@ import { readFile } from 'node:fs/promises';
 import { join } from 'node:path';
 import { createUserMessage } from '@deepseek-ai/dsh-llm';
 import { SessionLogOffset } from '@deepseek-ai/dsh-session';
+import { v3RenamedSourceKind } from "./context-source.js";
 import { memberMemoryDirectoryPath } from "./member-runtime.js";
 export const name = 'wowyuarm-agent-team-member-context';
 const MAX_MEMORY_BYTES = 8 * 1024;
@@ -42,8 +43,9 @@ export function apply(ctx) {
             // number domains explicit (the seq = log.length contiguity contract).
             const event = agent.session.snapshotEvents(SessionLogOffset(sequence), SessionLogOffset(sequence + 1))[0];
             return event?.type === 'user/message'
-                && event.data.source.kind === 'plugin'
-                && event.data.source.plugin === name
+                // Both identities are this producer's own: the kind written now and
+                // the read-time conversion's rename of this producer's V3 history.
+                && (event.data.source.kind === name || event.data.source.kind === v3RenamedSourceKind(name))
                 && event.data.content[0]?.type === 'text'
                 ? [event.data.content[0].text]
                 : [];
@@ -53,7 +55,7 @@ export function apply(ctx) {
             return decision;
         const message = createUserMessage({
             content: [{ type: 'text', text }],
-            source: { kind: 'plugin', plugin: name, form: 'instructions' },
+            source: { kind: name, form: 'instructions' },
         });
         return { kind: 'enter', messages: [...decision.messages, message] };
     }, { prepend: true });

package/packages/agent-team/lib/member-time-context.js

@@ -27,6 +27,7 @@
  * @module @wowyuarm/dsh-agent-team/member-time-context
  */
 import { createUserMessage } from '@deepseek-ai/dsh-llm';
+import { v3RenamedSourceKind } from "./context-source.js";
 import { formatTeamDuration, formatTeamTimestamp } from "./time-format.js";
 import { advanceOwnedSessionEventCursor } from "./session-event-cursor.js";
 export const name = 'wowyuarm-agent-team-member-time-context';
@@ -49,7 +50,9 @@ export function applyClockEvent(state, event) {
             return state.openTurn === -1 ? state : { ...state, lastTurnInjectionTime: null, openTurn: -1 };
         case 'user/message': {
             const source = event.data.source;
-            const injected = source.kind === 'plugin' && source.plugin === name;
+            // Both identities are this producer's own: the kind written now and the
+            // read-time conversion's rename of this producer's V3 history.
+            const injected = source.kind === name || source.kind === v3RenamedSourceKind(name);
             const withMessage = state.lastMessageTime === event.time ? state : { ...state, lastMessageTime: event.time };
             if (!injected)
                 return withMessage;
@@ -134,7 +137,7 @@ export function apply(ctx, config = {}) {
         const text = renderClockSnapshot({ now, turn, step, previous });
         const message = createUserMessage({
             content: [{ type: 'text', text }],
-            source: { kind: 'plugin', plugin: name, form: 'snapshot', sections: [{ name, text }] },
+            source: { kind: name, form: 'snapshot', sections: [{ name, text }] },
         });
         return { kind: 'enter', messages: [...decision.messages, message] };
     }, { prepend: true });

package/packages/agent-team/lib/pressure-policy.js

@@ -17,7 +17,7 @@
 import { CONTEXT_WINDOW_EXCEEDED_CODE } from '@deepseek-ai/dsh-llm';
 import { createUserMessage } from '@deepseek-ai/dsh-llm';
 import { CONTEXT_PRESSURE_NOTICE_SUMMARY } from "./context-management.js";
-import { AGENT_TEAM_PLUGIN_ID } from "./context-source.js";
+import { AGENT_TEAM_PLUGIN_ID, isAgentTeamSourceKind } from "./context-source.js";
 import { advanceOwnedSessionEventCursor } from "./session-event-cursor.js";
 /**
  * Whether this Session's own span already carries the pressure notice: either
@@ -50,8 +50,10 @@ export function contextPressureNoticeText(input) {
 }
 /** Whether one user message is this policy's one-shot pressure notice. */
 function isPressureNotice(message) {
+    // The source arrives untyped from the inbox; identity is matched by exact
+    // kind — the shape written now and the read-time conversion's V3 rename.
     const source = message.source;
-    return source?.plugin === AGENT_TEAM_PLUGIN_ID
+    return isAgentTeamSourceKind(source?.kind)
         && source?.summary === CONTEXT_PRESSURE_NOTICE_SUMMARY;
 }
 export class PressurePolicyCoordinator {
@@ -129,7 +131,7 @@ export class PressurePolicyCoordinator {
                             activeClaims: this.options.activeClaimLabels(member.memberId),
                             runningJobs: this.options.runningJobLabels(member.memberId),
                         }) }],
-                source: { kind: 'plugin', plugin: AGENT_TEAM_PLUGIN_ID, form: 'notice', summary: CONTEXT_PRESSURE_NOTICE_SUMMARY },
+                source: { kind: AGENT_TEAM_PLUGIN_ID, form: 'notice', summary: CONTEXT_PRESSURE_NOTICE_SUMMARY },
             });
             try {
                 agent.steer(notice);

package/packages/agent-team/lib/stored-session-reader.js

@@ -8,8 +8,7 @@
  * format generation, or Harness error message text.
  *
  * The reader owns reading and classification only — not Member lifecycle,
- * Agent create/resume, UI, ledger, or remediation (which reads raw artifact
- * bytes through its own repair path).
+ * Agent create/resume, UI, or ledger.
  *
  * Classification notes: the shipped JSONL backend throws its
  * `corrupt session log` family as plain `Error`s, so corruption is detected by