Skip to content

feat: Add environment hygiene and feature integration guidance to worker#307

Merged
dlorenc merged 1 commit intodlorenc:mainfrom
aronchick:upstream/worker-hygiene-integration
Jan 28, 2026
Merged

feat: Add environment hygiene and feature integration guidance to worker#307
dlorenc merged 1 commit intodlorenc:mainfrom
aronchick:upstream/worker-hygiene-integration

Conversation

@aronchick
Copy link
Copy Markdown
Contributor

Summary

Extends worker template with two new capability sections for better operational hygiene and PR integration workflows.

Environment Hygiene

  • Shell history stealth (leading space prefix for sensitive commands)
  • Pre-completion cleanup (verify no credentials leaked)

Feature Integration Tasks

  • Reuse First principle - search before writing new code
  • Minimalist Extensions guidance - add minimum necessary
  • PR analysis workflow (gh pr view, gh pr diff)
  • Integration checklist

Design Decision

Rather than creating separate agent types ("Sanitized-PR-Worker" and "Feature-Integration-Worker"), these capabilities are integrated into the existing worker template. This:

  • Avoids agent type proliferation
  • Makes capabilities available to ALL workers
  • Follows the 'reuse first' principle

Test Plan

  • Build succeeds: go build ./...
  • Templates tests pass: go test ./internal/templates/...

Fixes #282
Fixes #283

🤖 Generated with Claude Code

Extends worker template with two new capability sections:

**Environment Hygiene:**
- Shell history stealth (leading space prefix)
- Pre-completion cleanup (verify no credentials leaked)

**Feature Integration Tasks:**
- Reuse First principle
- Minimalist Extensions guidance
- PR analysis workflow
- Integration checklist

Follows the concise style of the existing worker template.

Fixes dlorenc#282
Fixes dlorenc#283

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@aronchick aronchick marked this pull request as ready for review January 25, 2026 19:23
@aronchick
Copy link
Copy Markdown
Contributor Author

Merge Queue Evaluation: ✅ Approved

Status: Ready for upstream maintainer to merge

CI: ✅ All checks passing
Scope: ✅ Matches title (36 additions, worker.md only)
Roadmap: ✅ Supports P0 'Reliable worker lifecycle'
Reviews: ✅ No blocking issues

This PR adds worker hygiene and feature integration guidance to the worker template. Small, focused, and roadmap-aligned.

Recommendation: Approve and merge when ready.

@aronchick
Copy link
Copy Markdown
Contributor Author

✅ APPROVED FOR MERGE

Merge Queue Decision: This PR is approved and ready for upstream maintainer to merge.

Status:

  • ✅ CI: All checks passing
  • ✅ Scope: Matches title (36 additions, worker.md only)
  • ✅ Roadmap: Supports P0 'Reliable worker lifecycle'
  • ✅ Reviews: No blocking issues
  • ✅ No conflicts

Upstream maintainer: This PR is ready to merge whenever you're ready.

@aronchick
Copy link
Copy Markdown
Contributor Author

Merge Queue Approval

This PR has been reviewed and is ready to merge:

  • CI: All 6 jobs passing
  • Reviews: No blocking reviews
  • Scope: Matches title (worker guidance addition)
  • ROADMAP: Aligns with P0 (reliable worker lifecycle)

Recommended action: Merge via squash

🤖 Approved by merge-queue agent

@dlorenc dlorenc merged commit ff1ab5c into dlorenc:main Jan 28, 2026
6 checks passed
aronchick added a commit to aronchick/multiclaude that referenced this pull request Jan 29, 2026
…ker (dlorenc#307)

Extends worker template with two new capability sections:

**Environment Hygiene:**
- Shell history stealth (leading space prefix)
- Pre-completion cleanup (verify no credentials leaked)

**Feature Integration Tasks:**
- Reuse First principle
- Minimalist Extensions guidance
- PR analysis workflow
- Integration checklist

Follows the concise style of the existing worker template.

Fixes dlorenc#282
Fixes dlorenc#283

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
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.

The Feature-Integration-Worker Sanitize sholl history

2 participants