Skip to content

Expand popular framework route extraction - #234

Merged
forhappy merged 1 commit into
mainfrom
codex/expand-popular-framework-routes
Aug 13, 2026
Merged

Expand popular framework route extraction#234
forhappy merged 1 commit into
mainfrom
codex/expand-popular-framework-routes

Conversation

@forhappy

Copy link
Copy Markdown
Contributor

Summary

  • add Angular Router typed/provider route configs, Echo and Fiber grouped routes, and ASP.NET Core Minimal API route groups
  • compose imported Express, Hono, and Fastify routers across modules through framework-owned static resolver adapters
  • preserve exact handlers and anchors while leaving ambiguous mounts local, enforcing explicit limits, and publishing atomically
  • advance extraction semantics to v11 and document the expanded route surface

Motivation

Compass already recognized many framework-local registrations, but popular ecosystems still had important gaps: Angular configs, Echo/Fiber routes, Minimal APIs, and cross-module Node router mounts. This change fills that first cross-language slice without adding central framework dispatch branches or allowing one framework adapter to mutate another.

Verification

cargo fmt --all -- --check
cargo clippy --workspace --lib --bins --locked -- -D warnings
cargo test --workspace --lib --bins --locked
cargo test -p compass-resolve --test native_routes --test typescript_routes --locked
./scripts/qualify_code_graph_v1.sh --fixtures-only

All commands above pass. The broader pre-existing cargo clippy -p compass-resolve --all-targets test lint remains blocked by unrelated expect/panic and collapsible-if violations in framework_qualification.rs and python_import_provenance.rs; the required workspace lib/bin lint passes.

Compatibility and documentation

The public graph and CLI schemas are unchanged. Extraction semantics advance from v10 to v11 so cached facts rebuild and expose the new additive route evidence. docs/reference/framework-routes.md and CHANGELOG.md describe the supported shapes.

Checklist

  • The change is focused and excludes unrelated formatting or generated files
  • Tests cover changed behavior, or this pull request changes documentation only
  • User-facing commands, flags, limits, and examples are documented
  • Compatibility or migration effects are described
  • No credentials, private source code, or sensitive report details are included
  • I agree to license my contribution under MIT OR Apache-2.0
  • I followed the Compass code of conduct

@forhappy
forhappy merged commit ec8a899 into main Aug 13, 2026
14 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