Repository navigation
Record only the project's tools' runs, list a format's answers, say test kinds once - #177
Merged
Merged
Conversation
…est kinds once A shared tool is one Tool per profile that adopts it, so one command became one ToolRun fact per profile and every fixture's facts grew with each profile sharing a tool it ran (cpp-best-practices#142). The tool-run stage now asks only the tools of the project's own languages and formats: a run of another language's tool answers nothing a question asks. The facts snapshots lose those runs; no result changes. --answers lists a format's answers to the tool-based questions it is asked (tests, formatting, analysis, security, audit, outputs, schema validation) where some tool could give one, so the YAML every project holds adds no line. tests-run's "no tests found" message joined each runner's description, so runners sharing kinds repeated them (cpp-best-practices#144); a Definition carries its kinds, and the testing and fuzzing policies say each kind once. The README, architecture and history documents say what the roadmap stack added. Closes cpp-best-practices#142. Closes cpp-best-practices#144. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01ByTXNqDWdM7HzezrFtEvQt
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #177 +/- ##
=======================================
Coverage 99.76% 99.76%
=======================================
Files 251 251
Lines 13266 13276 +10
Branches 1474 1475 +1
=======================================
+ Hits 13235 13245 +10
Misses 31 31 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
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.
Housekeeping after the roadmap stack (#176): two snapshot-hygiene issues, a listing gap, and the docs.
Only the project's tools record runs (#142)
A shared tool is one
Toolper profile that adopts it (kcov for Zig and Shell, prettier for JavaScript and JSON Schema, the misconfiguration scanners for Terraform, Docker, Kubernetes and Ansible), so one command became oneToolRunfact per profile, and every fixture's facts grew with each profile that shared a tool it ran. The tool-run stage now asks only the tools of the project's own languages and formats: a run of another language's tool answers nothing a question asks (questions are asked of the project's languages only). The facts snapshots lose those runs, about 17,000 lines across the fixtures; no result changes. The issue's first option, oneToolper program with the union of languages, remains available, but the churn it describes is gone. Four tests that built a project from a CI file alone and looked for a Go, JavaScript, Ruby or C tool's run now give that project a source of the language; one that listed the facts of a YAML-only project no longer expects ctest's run.A format's answers in
--answersThe tool-based questions a format is asked (tests, formatting, analysis, security, audit, outputs, schema validation) have
kinds=EVERY_KIND, and--answerslists a format's answer only where some unit has one a tool could give. So a Dockerfile's static analysis, a chart's tests and a notebook's formatting are listed, and the YAML every project holds, which nothing tests or scans, adds no line. Languages are listed as before.The test kinds said once (#144)
tests-run's "no tests found" message joined each runner's description, so runners sharing kinds repeated them (Erlang's bare fixture listed EUnit and Common Test twice, and differently for Elixir and Erlang). ADefinitioncarries itskindsbeside its joined description,KnownToolpasses them on, and the testing and fuzzing policies join the set of kinds once, witheither("A, B or C"). The BEAM and SQL bare fixtures' messages move accordingly, and Elixir's and Erlang's, now the same, are said once.Docs
The README's recognition list gains
source_namesand a probe that returns formats; the architecture document's structure, roles, tools and questions paragraphs say what the roadmap added (recognition by name and by content, thetemplaterole, which tool runs are recorded, which answers are listed); the history gains the roadmap stack as its twelfth step.🤖 Generated with Claude Code
https://claude.ai/code/session_01ByTXNqDWdM7HzezrFtEvQt