Skip to content

[v0.8 Prompts P02] add clear direct prose guidance - #1237

Open
sethkarten wants to merge 3 commits into
v080/promptsfrom
v080/prompts-p02-clear-direct-prose
Open

[v0.8 Prompts P02] add clear direct prose guidance#1237
sethkarten wants to merge 3 commits into
v080/promptsfrom
v080/prompts-p02-clear-direct-prose

Conversation

@sethkarten

@sethkarten sethkarten commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

P02 — clear direct prose guidance (child of P01)

Published main-topology child head: 6839d49bce9cadb463aaa3004e51ca65b0fc7d15
Base: v080/prompts at P01 head 1a53c7064c8689f39c56da4639be7df1738fbc8e.
Ordered parents: prior P02 d46d605ce56bfaa126d138d09db60d9b49dc72de, then main-based P01 1a53c7064c8689f39c56da4639be7df1738fbc8e.
Incremental scope: only packages/coding-agent/test/system-prompt.test.ts; raw reviewed P02 patch preserved byte-for-byte (SHA-256 aa3155112eda411030bd6ff5ea291c4cfdd719fa0b002e30d0d8bab981f04dc3).

Adds the literal built-in clear-direct-prose block and deterministic full-prompt coverage. Typed P01 provenance ensures custom including empty/lookalike content receives none and unknown fails closed. Independent Terra review accepted the final child tree; neutral validation passes 59/59 and root type-check passes. Merge only after P01.

Note

Add strict full-string equality assertion to buildRlmPrompt system prompt test

Replaces substring containment checks in system-prompt.test.ts with a single toBe assertion against the complete expected prompt string, covering all guidance sections and runtime context lines. Imports DEFAULT_RLM_EXTRA_IMPORT_LABELS from the bootstrap module to dynamically build the pre-installed packages line in the expected output. Risk: any change to the prompt text, including whitespace or ordering, will now fail the test.

Macroscope summarized 6839d49.


Note

Cursor Bugbot is generating a summary for commit 6839d49. Configure here.

@sethkarten
sethkarten marked this pull request as ready for review August 11, 2026 18:23
@sethkarten
sethkarten requested review from alexzhang13, skirsten and snimu and removed request for skirsten August 11, 2026 18:23
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