test: compare a fresh full baseline with PR #5 incremental analysis - #7
Draft
ivanmilevtues wants to merge 2 commits into
Draft
test: compare a fresh full baseline with PR #5 incremental analysis#7ivanmilevtues wants to merge 2 commits into
ivanmilevtues wants to merge 2 commits into
Conversation
Amp-Thread-ID: https://ampcode.com/threads/T-01a08203-82f5-7539-bc7f-7c6512cf51b8 Co-authored-by: Ivan Milev <ivanmilevtues@gmail.com>
Member
Author
Full-analysis Mermaid — comparison with PR #5Generated directly from the committed Top level — same depth as the PR #5 commentgraph LR
n_1["owner"]
n_2["vet"]
n_3["system"]
n_4["model"]
n_5["Loose files in org.springframework.samples.petclinic"]
n_1 -- "calls" --> n_4
Expanded — all stored components and relationshipsgraph LR
subgraph n_1["owner"]
n_1_1["Owner"]
n_1_2["PetController"]
n_1_3["Pet"]
n_1_4["Visit"]
end
n_2["vet"]
n_3["system"]
n_4["model"]
n_5["Loose files in org.springframework.samples.petclinic"]
n_1_1 -- "calls" --> n_1_3
n_1_1 -- "calls" --> n_4
n_1_2 -- "calls" --> n_1_1
n_1_2 -- "calls" --> n_1_3
n_1_2 -- "calls" --> n_4
n_1_3 -- "calls" --> n_4
n_1_4 -- "calls" --> n_1_1
n_1_4 -- "calls" --> n_1_3
The eight stored relationships collapse to one cross-component edge ( Compare with PR #5’s incremental diagram. The top-level names and edge match; PR #5 additionally marks Full-analysis run · same application source, action revision, engine 0.14.0, hosted provider, and depth 2 as the original run. |
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.
Full-analysis comparison with #5
A fresh full baseline has been generated and committed. This draft PR targets
refactor/vet-directory-review-demo, notmain, and contains the same application source as #5..codeboarding/analysis.jsonin this PRControls and verification
The full-analysis workflow pins the exact action revision from the original review, including engine 0.14.0, and uses the same hosted provider and depth 2. It runs
mode: syncwithforce_full: trueon this comparison branch only. The moving v1 tag has changed since the original run, so it is deliberately not used here.Logs confirm
Running FULL analysis workflowandForce full analysis: skipping static analysis cache. A source diff against the original PR branch is empty. The original artifact metadata recordsmode: incrementalandseed_source: base.Both outputs have the same five top-level component names and eight component relations; that is a coarse comparison, not a claim that the complete graphs are identical. Hosted model responses can vary between runs.
Left as a draft to avoid an unrelated incremental review of generated baseline files. No application code changes, no changes to main, and no modifications to the original review result. This PR is for comparison, not merging.