feat(senpi-codemode): teach eval batching via instruction and reuse-chain examples#205
Merged
Merged
Conversation
…hain examples Replace the json-config example payloads with a three-cell chain that demonstrates comprehension filtering, batch read() loops, and parallel() fan-out of tool.<name>() calls. Add a 'Batch through eval' instruction absorbing the old parallelize sentence, drop the rb example slots (rb defaults off), and delete the critical-block sentence duplicated verbatim in the instruction.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Rework the eval tool prompt so batch-through-eval usage surfaces on its own instead of relying on operator steering.
**Batch through eval.**paragraph — one cell replaces many one-shot tool calls: loop/comprehend over file sets withread()/stdlib, post-processtool.<name>()results programmatically, fan independent calls throughparallel(thunks). Absorbs the old "Parallelize work within a cell" sentence (replacement, not addition).rglobcollection → batchread()loop withCounter→parallel()fan-out oftool.grepcalls, while keeping the cell-to-cell state-reuse narrative intact. The two rb slots are dropped — rb defaults off, so the default profile never rendered them.<critical>sentence duplicated verbatim from the instruction ("Re-run setup only after reset/crash/NameError") is deleted.Net: prompt teaches harness-specific, non-derivable patterns (thunk shape with
d=dbinding,tool.<name>args dict, batch-read idiom) in the same slot count; file-mutation and bash payloads intentionally excluded to avoid steering around the edit tool or double-bridging shell.QA
npx tsx vitest --run test/inpackages/senpi-codemode: 350 passed / 6 skipped (snapshots updated deliberately, reviewed line-by-line).npm run checkat repo root: passed (biome, docs, types, neo build+vet+test).Summary by cubic
Teaches eval batching in
senpi-codemodewith a new instruction and Python reuse-chain examples showing batch reads and parallel tool calls. Removes Ruby examples and duplicate guidance; prompt-only change with updated tests.rglob, batchread()+Counter, and paralleltool.grepfan-out using thunks.Written for commit 31936d8. Summary will update on new commits.