Repository navigation
refactor(architecture): resolve the view medium through the shared resolver #13027
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
18 commits
Select commit
Hold shift + click to select a range
0fd08fb
Resolve issue #12672
melodic-automation-lanes[bot] adec27b
Fix failing CI checks on work-resolve-issue/issue-12672-41723143585-1
melodic-automation-lanes[bot] c371e77
Merge main into work-resolve-issue/issue-12672-41723143585-1
melodic-automation-lanes[bot] 0893087
Merge remote-tracking branch 'origin/work-resolve-issue/issue-12672-4…
kyle-sexton 504eff8
refactor(architecture): major bump and retired-file wording for the m…
kyle-sexton ee680d2
Merge origin/main
kyle-sexton 35e29cd
fix(architecture): say what to do when the resolver rejects a named m…
kyle-sexton abf354f
Merge remote-tracking branch 'origin/main' into fix/12991-replacement
kyle-sexton 33e8dd6
Merge origin/main
kyle-sexton 9dd9406
Merge main into fix/12991-replacement
melodic-automation-lanes[bot] 612522f
Merge main into fix/12991-replacement
melodic-automation-lanes[bot] e83afa1
chore: start the lane checks again on main's workflow files
melodic-automation-lanes[bot] a132c08
Merge main into fix/12991-replacement
melodic-automation-lanes[bot] 9f7d1ee
Merge origin/main into fix/12991-replacement
kyle-sexton 6c4c2c6
Merge main into fix/12991-replacement
melodic-automation-lanes[bot] c0f17f1
Merge remote-tracking branch 'origin/main' into HEAD
kyle-sexton 5eef443
fix(architecture): drop the architecture row from the view-medium exe…
kyle-sexton 17f4260
Merge origin/main
kyle-sexton File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
15 changes: 15 additions & 0 deletions
15
.changes/architecture/work-resolve-issue-issue-12672-41723143585-1-6568f049.md
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,15 @@ | ||
| --- | ||
| bump: major | ||
| --- | ||
|
|
||
| ### Added | ||
|
|
||
| - A `medium` plugin option that sets where the architecture views go. It declares `options`, so the plugin now needs Claude Code v2.1.271 or later ([Limit a field to fixed options](https://code.claude.com/docs/en/plugins/manifest-reference#limit-a-field-to-fixed-options)). | ||
|
|
||
| ### Changed | ||
|
|
||
| - The `map-*` views and the `improve` deepening report now resolve their medium through the shared resolver (`lib/view-settings.mjs resolve --lane architecture`) and print its report line. | ||
|
|
||
| ### Removed | ||
|
|
||
| - `~/.claude/rendered-views.md` and the `.claude/rendered-views*.md` files no longer choose a publishing medium: an `artifact` or `hosted` value there is ignored. A `terminal` or `file` value there still keeps the view local when no scope sets a value. Each file gives one notice per project naming where to set the medium instead: this plugin's `medium` option, `docs/conventions/rendered-views.yaml`, or `docs/conventions/rendered-views.local.yaml`. |
7 changes: 7 additions & 0 deletions
7
.changes/debugging/work-resolve-issue-issue-12672-41723143585-1-124dbf76.md
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,7 @@ | ||
| --- | ||
| bump: patch | ||
| --- | ||
|
|
||
| ### Changed | ||
|
|
||
| - The shared `reference/rendered-view.md` now lists `architecture` as a medium-resolver lane beside `planning`; the post-mortem view still reads the rendered-views cascade itself. |
7 changes: 7 additions & 0 deletions
7
.changes/discovery/work-resolve-issue-issue-12672-41723143585-1-b0a6ab28.md
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,7 @@ | ||
| --- | ||
| bump: patch | ||
| --- | ||
|
|
||
| ### Changed | ||
|
|
||
| - The shared `reference/rendered-view.md` now lists `architecture` as a medium-resolver lane beside `planning`; the blindspot view still reads the rendered-views cascade itself. |
7 changes: 7 additions & 0 deletions
7
.changes/planning/work-resolve-issue-issue-12672-41723143585-1-40719358.md
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,7 @@ | ||
| --- | ||
| bump: patch | ||
| --- | ||
|
|
||
| ### Changed | ||
|
|
||
| - The shared `reference/rendered-view.md` now lists `architecture` as a medium-resolver lane beside `planning`, and adds the architecture deepening report's medium rules. |
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
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
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
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.