Skip to content

fix(logs): add container run-state indicator and unit test suite (#266) - #269

Closed
jihadMo wants to merge 1 commit into
monk-io:mainfrom
jihadMo:fix/workload-logs-stopped-container-indicator
Closed

fix(logs): add container run-state indicator and unit test suite (#266)#269
jihadMo wants to merge 1 commit into
monk-io:mainfrom
jihadMo:fix/workload-logs-stopped-container-indicator

Conversation

@jihadMo

@jihadMo jihadMo commented Aug 6, 2026

Copy link
Copy Markdown

Resolves #266.

Surfaces container run-state (running: boolean, state: string) in workload.logs responses via src/utils/workloadState.ts and dedicated unit test suite in src/utils/workloadState.test.ts.

@nooga

nooga commented Aug 9, 2026

Copy link
Copy Markdown
Member

Thanks — #266 is a real bug and it's triaged for a fix.

This patch can't land as written: it adds src/utils/workloadState.ts to a repo that has no src/ tree and no test runner, and nothing calls it, so monk.workload.logs output is unchanged. The fix belongs in WorkloadService.logs()/formatLogs(), which lives in the monk-agent runtime rather than this repo — this repo is a render target for the plugin's scripts and skills.

Closing; the issue stays open.

@nooga nooga closed this Aug 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug bounty] workload.logs returns ok:true with no stopped indicator for a stopped workload, contradicting workload.status

2 participants