Skip to content

chore: release - merge dev into main#414

Merged
zbigniewsobiecki merged 9 commits intomainfrom
dev
Feb 18, 2026
Merged

chore: release - merge dev into main#414
zbigniewsobiecki merged 9 commits intomainfrom
dev

Conversation

@zbigniewsobiecki
Copy link
Copy Markdown
Member

Automated release PR created by the release workflow.

Commits (9):

820b9db Merge pull request #411 from zbigniewsobiecki/feature/progress-comment-replace
16b9325 fix(progress): wrap state file cleanup in try-catch in stop()
597fce5 fix(progress): write state file from first tick when initial comment fails
e923f93 Merge pull request #410 from zbigniewsobiecki/refactor/agent-lifecycle-dedup
02d5e8e refactor(agents): make capabilities.ts single source of truth for AgentCapabilities
e91c930 fix(progress): wire repoDir to call sites and fix API asymmetry
d77f5ce refactor(agents): wire up AgentCapabilities and remove duplication
5df9dc7 feat(progress): replace initial progress comment with final summary via state file
728fd4f refactor(agents): extract shared lifecycle utilities to eliminate code duplication

Cascade Bot and others added 9 commits February 18, 2026 18:19
- Extract AgentCapabilities interface and registry to
  agents/shared/capabilities.ts
- Replace hardcoded agentType checks in base.ts with
  capability lookups
- Remove needsGitHubToken from AgentCapabilities
  (top-level AgentProfile field is single source of truth)
- Update adapter test to match simplified capabilities

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Pass repoDir to createProgressMonitor in adapter.ts and base.ts so the
  state file is actually written (previously dead code)
- Update lifecycle.ts callback signature to forward repoDir
- Add optional repoDir param to readProgressCommentId() for API symmetry
  with writeProgressCommentId() and clearProgressCommentId()
- Add .cascade-progress-comment-id to .gitignore
- Merge split fs imports in progressState.ts
- Replace require('node:fs') with ESM imports in progressState tests
- Add clarifying comment on maybeWriteStateFile call path

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…ntCapabilities

Replace inline capability objects in agent-profiles.ts with getAgentCapabilities()
lookups, eliminating the dual-registry sync risk flagged in code review.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…e-dedup

refactor(agents): extract shared lifecycle utilities to eliminate code duplication
…fails

The `else` branch in `postProgressToPM` (entered when `postInitialComment()`
fails and the first tick creates the comment) was not calling
`maybeWriteStateFile()`, leaving the state file unwritten. This meant
the PostComment gadget would not find the progress comment ID and would
create a duplicate instead of replacing it.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
clearProgressCommentId() uses rmSync which can throw on permissions
errors. Since stop() is called from finally blocks in lifecycle.ts
and adapter.ts, an uncaught exception would mask the actual agent
execution result. Wrap in try-catch for best-effort cleanup.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…t-replace

feat(progress): replace initial progress comment with final summary via state file
@zbigniewsobiecki zbigniewsobiecki merged commit db44fca into main Feb 18, 2026
9 checks passed
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.

1 participant