Conversation
Introduce a new `testing.toml` knowledge section covering four rules for the `dev` persona's assistant: forcing and proving exceptional paths in tests, never weakening a failing assertion instead of diagnosing it, asserting user-visible output exactly rather than with `contains()`, and knowing the delivery semantics of concurrency/IPC primitives before building tests on top of them. The `dev` persona now extends this file alongside its existing knowledge sources. Also simplify the `rfd-implementor` persona's phase-discipline instructions: drop the explicit "phase 1" reference in favor of "the first phase", and collapse the per-phase verification guidance to run `cargo_format`, `cargo_check`, and the full `cargo_test` command, instead of separately calling out targeted `cargo_test` runs. Signed-off-by: Jean Mertz <git@jeanmertz.com>
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
1 participant
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.
Introduce a new
testing.tomlknowledge section covering four rules for thedevpersona's assistant: forcing and proving exceptional paths in tests, never weakening a failing assertion instead of diagnosing it, asserting user-visible output exactly rather than withcontains(), and knowing the delivery semantics of concurrency/IPC primitives before building tests on top of them. Thedevpersona now extends this file alongside its existing knowledge sources.Also simplify the
rfd-implementorpersona's phase-discipline instructions: drop the explicit "phase 1" reference in favor of "the first phase", and collapse the per-phase verification guidance to runcargo_format,cargo_check, and the fullcargo_testcommand, instead of separately calling out targetedcargo_testruns.