Skip to content

[Midtown !4] Tighten project-lead 17.2.5#2333

Merged
btucker merged 1 commit intomainfrom
task-4-tighten-midtown-project-lead-agent
Apr 7, 2026
Merged

[Midtown !4] Tighten project-lead 17.2.5#2333
btucker merged 1 commit intomainfrom
task-4-tighten-midtown-project-lead-agent

Conversation

@btucker
Copy link
Copy Markdown
Owner

@btucker btucker commented Apr 7, 2026

Summary

  • Enforces spec 17.2.5: the project lead must not investigate before delegating
  • Replaces the "fork into thread for research" escape hatch with an explicit prohibition against reading files, searching the codebase, or forking for research prior to delegation
  • Makes clear the coworker owns investigation as part of their task

Test plan

  • Verify project lead no longer forks for research before creating tasks
  • Confirm delegation happens immediately on receipt of investigation-type requests

🌃 Co-built with Midtown

Per spec 17.2.5: the project lead must delegate immediately without
front-loading investigation (file reads, codebase search, or fork for
research). Replaces the "fork into thread" escape hatch with an explicit
prohibition.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: e15b322b40

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

When you receive a user message, promptly respond with `@user` to acknowledge and briefly explain what you plan to do. This gives the human immediate feedback rather than silence while you work on delegation.

If the request needs investigation (code exploration, debugging, task scoping), **fork into the thread** after acknowledging. This keeps you available for other messages while the fork handles the research.
**Delegate immediately — do not investigate first.** When the request involves code exploration, debugging, or task scoping, create a task for a coworker to handle it. Do not read files, search the codebase, or fork for research before delegating. The coworker performs the investigation as part of their work. Your role is to route and frame the task, not to front-load discovery yourself.
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Preserve lead-driven path before mandating delegation

This new instruction is unconditional, but lead-driven channels are the explicit exception where the lead is expected to do the work directly and auto-dispatch is disabled (src/daemon_v2/decisions/dispatch.rs skips spawning when is_lead_driven is true). In that configuration, telling the project lead to always create a coworker task for investigation can leave requests stalled because no coworker is auto-assigned; add a lead-driven exception (or allow direct/fork investigation in that mode) so lead-driven workflows keep functioning.

Useful? React with 👍 / 👎.

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 63.97%. Comparing base (05897b0) to head (e15b322).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2333      +/-   ##
==========================================
+ Coverage   63.70%   63.97%   +0.27%     
==========================================
  Files         100      100              
  Lines       38582    38592      +10     
==========================================
+ Hits        24578    24691     +113     
+ Misses      14004    13901     -103     

see 9 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@btucker btucker merged commit ecd61f1 into main Apr 7, 2026
13 checks passed
@btucker btucker deleted the task-4-tighten-midtown-project-lead-agent branch April 7, 2026 03:50
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