Skip to content

Commit d77a69b

Browse files
committed
fix: address PR review comments on deploy output handling
1 parent a68d1c5 commit d77a69b

1 file changed

Lines changed: 8 additions & 1 deletion

File tree

src/cli/tui/screens/deploy/useDeployFlow.ts

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -161,7 +161,7 @@ export function useDeployFlow(options: DeployFlowOptions = {}): DeployFlowState
161161
}
162162
}
163163
if (Object.keys(outputs).length === 0) {
164-
logger.log('Warning: Could not retrieve stack outputs after polling', 'warn');
164+
throw new Error('Could not retrieve stack outputs after polling. Deployed state will not be recorded.');
165165
}
166166

167167
const agents = parseAgentOutputs(outputs, agentNames, currentStackName);
@@ -194,6 +194,13 @@ export function useDeployFlow(options: DeployFlowOptions = {}): DeployFlowState
194194
const memoryNames = (ctx.projectSpec.memories ?? []).map((m: { name: string }) => m.name);
195195
const memories = parseMemoryOutputs(outputs, memoryNames);
196196

197+
if (memoryNames.length > 0 && Object.keys(memories).length !== memoryNames.length) {
198+
logger.log(
199+
`Deployed-state missing outputs for ${memoryNames.length - Object.keys(memories).length} memory(ies).`,
200+
'warn'
201+
);
202+
}
203+
197204
// Expose outputs to UI
198205
setStackOutputs(outputs);
199206

0 commit comments

Comments
 (0)