Skip to content

test: add comprehensive integration tests and CI pipeline validation …#4

Merged
honeydew78 merged 1 commit into
mainfrom
feature
Jun 11, 2026
Merged

test: add comprehensive integration tests and CI pipeline validation …#4
honeydew78 merged 1 commit into
mainfrom
feature

Conversation

@honeydew78

Copy link
Copy Markdown
Owner

…for dependency impact analysis

@github-actions

Copy link
Copy Markdown

🛡️ Diff-Guard Architectural Risk Report

PR: #0 | Status: 🟡 MEDIUM RISK | Risk Score: 20/100

🔍 Modified Semantic Entities

File Entity Change Type
app.py analyze_diff_and_report() MODIFIED
app.py format_markdown_report() MODIFIED
app.py parse_github_url() MODIFIED
app.py api_get_branches() MODIFIED
app.py api_get_commits() MODIFIED
app.py api_analyze() MODIFIED
app.py _cached_fetch() ADDED
app.py compute_risk_score() ADDED
app.py _build_summary_text() ADDED
app.py run_analysis_pipeline() ADDED
app.py api_health() ADDED
tests/test_api.py test_health_endpoint() ADDED
tests/test_graph_engine.py test_shortest_path_finding() ADDED
tests/test_graph_engine.py test_find_functions_using_symbol_via_graph() ADDED
tests/test_parser.py _get_changed_line_numbers() ADDED
tests/test_parser.py _run_sandbox_analysis() ADDED
tests/test_parser.py test_sandbox_modified_file_detection() ADDED
tests/test_parser.py test_sandbox_compute_value_modified() ADDED
tests/test_parser.py test_sandbox_blast_radius() ADDED
tests/test_parser.py test_sandbox_impact_path() ADDED
tests/test_parser.py test_sandbox_at_risk_function() ADDED

🕸️ Architectural Blast Radius

⚠️ 2 downstream files are directly or transitively affected:

Modified File Impacted Downstream Files (Dependency Path)
app.py app.pytests/test_api.py
app.py app.pycli.py
tests/test_api.py None (Terminal module)
tests/test_graph_engine.py None (Terminal module)
tests/test_parser.py None (Terminal module)

🔍 Downstream Functions at Risk

The following specific functions in direct consumer files rely on the modified symbols and may be at risk:

Consumer File At-Risk Functions
cli.py run_analysis()

@honeydew78 honeydew78 merged commit f8076f3 into main Jun 11, 2026
2 checks passed
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.

1 participant