Remove scope labels from context output and update README example#80
Merged
Conversation
…lobal context example - Context files in --elaborate output are now listed as plain paths without [global]/[local] scope prefixes - Global context files still appear before local context files - Add top-level context: example to README .reviewmark.yaml schema - Update tests and docs to match new plain-path output Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…rification docs Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request simplifies how context files are presented in ReviewMark elaboration output and documentation by removing [global]/[local] scope labels and listing context entries as plain file paths (with global context shown before per-review-set context).
Changes:
- Updated
ElaborateReviewSetto emit context entries as plain paths (no scope labels), with global context files listed first. - Updated documentation and requirements text to describe the new context display format.
- Updated tests and README example configuration to match the new output format.
Reviewed changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| test/DemaConsulting.ReviewMark.Tests/Configuration/ReviewMarkConfigurationTests.cs | Updates assertions/descriptions for context output; needs stronger assertions for the new format and ordering. |
| src/DemaConsulting.ReviewMark/Configuration/ReviewMarkConfiguration.cs | Removes scope-labeling logic and outputs plain context paths with global-first ordering. |
| README.md | Updates example .reviewmark.yaml to include top-level context: and reflect new behavior. |
| docs/verification/review-mark/configuration/review-mark-configuration.md | Updates verification expectations to “plain path” context entries. |
| docs/user_guide/introduction.md | Updates example --elaborate output to list context as plain paths. |
| docs/reqstream/review-mark/configuration/review-mark-configuration.yaml | Updates requirement text to specify plain paths and global-first ordering. |
| docs/design/review-mark/configuration/review-mark-configuration.md | Updates design documentation for ElaborateReviewSet context output behavior. |
| docs/design/review-mark/configuration.md | Updates configuration design overview to reflect plain-path context listing. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Move block-level evidence-source and context comments to own lines - Remove redundant inline comment from per-review-set context: - Normalise inline comment alignment to column 26 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Assert exact list-item format (- \path\) not just substring contains - Assert DoesNotContain [global]/[local] labels in both context tests - Add GlobalContextBeforeLocalContext test to verify ordering Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
This was referenced Jun 24, 2026
This was referenced Jun 29, 2026
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.
This pull request updates the way context files are displayed in review documentation and related outputs. Instead of labeling context files as
[global]or[local], the files are now listed as plain paths, with global context files appearing first, followed by per-review-set files. This change is reflected in the documentation, implementation, and tests to ensure consistency throughout the codebase.Documentation updates:
[global]and[local]labels. [1] [2] [3] [4] [5]Implementation changes:
ElaborateReviewSetinReviewMarkConfiguration.csto output context files as plain paths, with global files listed before per-review-set files, and removed the labeling logic.Test updates:
[global]and[local]labels. [1] [2] [3]Example and configuration updates:
README.mdand YAML configuration to match the new context file display format.These changes ensure a simpler and more consistent presentation of context files across documentation, code, and tests.