Skip to content

chat: shorter developer-friendly replies - #1428

Merged
AbirAbbas merged 4 commits into
devfrom
codex/todo-concise-replies
Sep 30, 2026
Merged

AbirAbbas merged 4 commits into
devfrom
codex/todo-concise-replies

Conversation

@ZeroPoint95

@ZeroPoint95 ZeroPoint95 commented Sep 23, 2026 •

Copy link
Copy Markdown
Collaborator

Changelog

  • Keep routine coding replies short and outcome-first; honor requested length and format.
  • Keep checks, failures, next actions and complete deliverables visible.
  • Shorten the guidance by 192 bytes without raising prompt budgets; restore no closing offer for the worker-isolation check.

Checks

  • Rebased onto latest dev; preserved interim updates.
  • All three review regression tests and make pr-ready passed.

@santoshkumarradha santoshkumarradha added feature Work that adds a capability; developers break it into tasks area:chat The v3 surface a person sits in front of (internal/tui3) labels Sep 27, 2026
@santoshkumarradha santoshkumarradha added this to the Chat & connections milestone Sep 27, 2026
@AbirAbbas
AbirAbbas marked this pull request as ready for review September 30, 2026 15:42
@AbirAbbas

Copy link
Copy Markdown
Collaborator

Heads up before this merges: on current dev (f769be4) this PR fails two internal/session laws that landed after its CI ran. Dev alone passes both.

  • TestTheFixedPrefixStaysUnderItsBudget: fixed prefix 57,380 bytes, over its 57,218 budget by 162. TestTheLeanPrefixStaysUnderItsBudget: 49,615, over 49,590 by 25. Dev had only 25 bytes of headroom, and the new report format adds 179 to system.md.
  • TestABeltWorkerReadsTheBeltsPagesAndNothingElse (prompt_belt_pages_test.go:58): "the sentinel "no closing offer" is no longer on prompts/system.md". The rewrite dropped that phrase, and the test uses it to prove belt workers don't get the chat's pages.

Either trim ~180 bytes from the prompt addition, or raise the waiver in prefixbudget_test.go together with PERF.md. Then keep that phrase, or move the test's sentinel to one the chat page still says. Repro: go test -count=1 -run '^(TestTheFixedPrefixStaysUnderItsBudget|TestTheLeanPrefixStaysUnderItsBudget|TestABeltWorkerReadsTheBeltsPagesAndNothingElse)$' ./internal/session/ on this branch merged with dev.

@ZeroPoint95
ZeroPoint95 force-pushed the codex/todo-concise-replies branch from 7cb2e25 to 8fef0b1 Compare September 30, 2026 16:41
@AbirAbbas
AbirAbbas merged commit db40bf9 into dev Sep 30, 2026
4 checks passed
@AbirAbbas
AbirAbbas deleted the codex/todo-concise-replies branch September 30, 2026 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:chat The v3 surface a person sits in front of (internal/tui3) feature Work that adds a capability; developers break it into tasks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants