Skip to content

Merge dev to main#481

Merged
zbigniewsobiecki merged 2 commits intomainfrom
dev
Feb 22, 2026
Merged

Merge dev to main#481
zbigniewsobiecki merged 2 commits intomainfrom
dev

Conversation

@zbigniewsobiecki
Copy link
Copy Markdown
Member

Summary

Routine dev → main promotion.

zbigniewsobiecki and others added 2 commits February 22, 2026 16:34
…ithout plan edits (#479)

The planning comment agent previously assumed every comment required plan
modifications, wasting iterations on unnecessary UpdateWorkItem calls when
users just asked questions like "Why this approach?" or "Can you explain
step 3?". Mirror the pattern from respond-to-pr-comment: classify the
comment as a question (PostComment only), a plan update (existing behavior),
or both, and act accordingly.

- Add Comment Classification section (Category A/B/C) to the prompt template
- Soften CRITICAL rules: "ONLY job" → "primary role", remove unconditional
  UpdateWorkItem mandate
- Add branching task flow: questions get PostComment-only, updates get plan
  edits + summary, mixed gets both
- Add question-only response format ("Re: [topic]") alongside existing
  "Plan Updated" format
- Update taskPrompts.ts buildCommentResponsePrompt with classification
  instruction matching buildPRCommentResponsePrompt pattern
- Add tests for new classification behavior and template rendering

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
…env vars directly (#480)

Credentials are immutable during a worker run. The previous three-mechanism
approach (JSON blob env var → parse + spread to process.env → populate cache
module → credential functions check env + cache + DB guards) is replaced by
a single mechanism: router sets individual env vars, credential functions
read from process.env.

- Router sets individual env vars + CASCADE_CREDENTIAL_KEYS sentinel
- Delete loadRouterCredentials() from worker-entry (env vars set by Docker)
- Credential functions use CASCADE_CREDENTIAL_KEYS as worker-context guard
- getAllProjectCredentials reconstructs map from env var key list
- Remove legacy GITHUB_TOKEN fallback in resolveGitHubToken
- Remove dead CASCADE_CREDENTIALS/CASCADE_CREDENTIALS_PROJECT_ID from scrub list

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
@zbigniewsobiecki zbigniewsobiecki merged commit e6890f1 into main Feb 22, 2026
10 checks passed
@zbigniewsobiecki zbigniewsobiecki deleted the dev branch February 22, 2026 16:22
@zbigniewsobiecki zbigniewsobiecki restored the dev branch February 22, 2026 16:53
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