Skip to content

feat(action): add composite GitHub Action for Pacta architecture review#17

Merged
akhundMurad merged 12 commits intomainfrom
feature/github-action
Jan 29, 2026
Merged

feat(action): add composite GitHub Action for Pacta architecture review#17
akhundMurad merged 12 commits intomainfrom
feature/github-action

Conversation

@akhundMurad
Copy link
Copy Markdown
Member

@akhundMurad akhundMurad commented Jan 28, 2026

  • extend DiffSummary to include added/removed node/edge names and serialize them
  • humanize diff node/edge identifiers when building summaries
  • add TrendPoint/TrendSummary and include trends in Report
  • introduce GitHubReportRenderer for rich GitHub PR comment output
  • add comprehensive tests for GitHub renderer output and table alignment

@akhundMurad
feat(reporting): add GitHub PR markdown renderer with trends and diff…
ce3a6c0 
… detail names

- extend `DiffSummary` to include added/removed node/edge names and serialize them
- humanize diff node/edge identifiers when building summaries
- add `TrendPoint`/`TrendSummary` and include trends in `Report`
- introduce `GitHubReportRenderer` for rich GitHub PR comment output
- add comprehensive tests for GitHub renderer output and table alignment
@akhundMurad
feat(cli): add GitHub report format with snapshot-based trends summary
37eb368
@akhundMurad
feat(action): add composite GitHub Action for Pacta architecture review
f6eac32
@akhundMurad
docs: add GitHub Action integration guide and document github output …
654c204 
…format
@akhundMurad akhundMurad self-assigned this Jan 28, 2026
@pacta-dev pacta-dev deleted a comment from github-actions bot Jan 28, 2026
@akhundMurad
feat(action): add target_dir input to control scan root
2f933de 
Update workflow example and use target_dir in pacta scan commands instead of hardcoded `.`
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jan 28, 2026 •
edited
Loading

Architecture Report

Commit: 3127c35

Violations (4 total)

Severity Count
Warning 1
Info 3

Violation Details

INFO infra_can_use_domain — Infrastructure implementing domain interfaces
src/infra/database.py:5 — "src.infra.database" in infra layer imports "src.domain.user" in domain layer

INFO infra_can_use_domain — Infrastructure implementing domain interfaces
src/infra/order_repository.py:3 — "src.infra.order_repository" in infra layer imports "src.domain.order" in domain layer

INFO infra_can_use_domain — Infrastructure implementing domain interfaces
src/infra/user_repository.py:5 — "src.infra.user_repository" in infra layer imports "src.domain.user" in domain layer

WARNING no_ui_to_infra — UI should not directly access Infrastructure
src/ui/user_controller.py:7 — "src.ui.user_controller" in ui layer imports "src.infra.database" in infra layer
Access infrastructure through application services instead

Architecture Trends (last 5 snapshots)

Metric Trend Current Change
Violations → Stable 4 +0
Modules ↓ Decreasing 15 -4
Dependencies ↓ Decreasing 12 -8
Density ↓ Decreasing 0.80 -0.25

Generated by Pacta v0.0.5

@akhundMurad akhundMurad merged commit 602d97c into main Jan 29, 2026
7 checks passed
@akhundMurad akhundMurad deleted the feature/github-action branch January 29, 2026 00:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@akhundMurad akhundMurad

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@akhundMurad