Skip to content

test: compare a fresh full baseline with PR #5 incremental analysis - #7

Draft
ivanmilevtues wants to merge 2 commits into
refactor/vet-directory-review-demofrom
test/full-analysis-pr5
Draft

test: compare a fresh full baseline with PR #5 incremental analysis#7
ivanmilevtues wants to merge 2 commits into
refactor/vet-directory-review-demofrom
test/full-analysis-pr5

Conversation

@ivanmilevtues

@ivanmilevtues ivanmilevtues commented Sep 8, 2026

Copy link
Copy Markdown
Member

Full-analysis comparison with #5

A fresh full baseline has been generated and committed. This draft PR targets refactor/vet-directory-review-demo, not main, and contains the same application source as #5.

Controls 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: sync with force_full: true on 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 workflow and Force full analysis: skipping static analysis cache. A source diff against the original PR branch is empty. The original artifact metadata records mode: incremental and seed_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.

Copy link
Copy Markdown
Member Author

Full-analysis Mermaid — comparison with PR #5

Generated directly from the committed analysis.json. This is the full snapshot, so no added/modified/deleted colors are applied.

Top level — same depth as the PR #5 comment

graph 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
Loading

Expanded — all stored components and relationships

graph 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
Loading

The eight stored relationships collapse to one cross-component edge (owner → model) at the top level; the others are internal to owner. The JSON contains no inter-component relationships for vet, system, or the loose-files component, so none are invented here.

Compare with PR #5’s incremental diagram. The top-level names and edge match; PR #5 additionally marks vet as modified.

Full-analysis run · same application source, action revision, engine 0.14.0, hosted provider, and depth 2 as the original run.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants