Skip to content

fix: exclude client navigation from inferred HTTP calls - #2519

Open
DeusData wants to merge 3 commits into
mainfrom
fix/issue-1250-client-navigation-v2
Open

DeusData wants to merge 3 commits into
mainfrom
fix/issue-1250-client-navigation-v2

Conversation

@DeusData

@DeusData DeusData commented Oct 3, 2026

Copy link
Copy Markdown
Owner

Client navigation calls such as router.navigateByUrl('/users'), location.go('/account') and $router.push('/home') could create HTTP_CALLS edges from the parallel resolver’s generic URL-argument fallback. Exclude known navigation spellings in JavaScript-family files while preserving explicit HTTP-client resolution, broker topics and other languages. Custom names such as navigateApi remain eligible for ordinary inference.

The regression fixture checks extraction and source nodes, absent navigation edges/routes, positive HTTP controls (including an axios import aliased as navigate), raw broker topics, and Python behavior. Query failures cannot masquerade as absent edges.

Depends on #2513 (URL/topic folding), including its formatting correction. This PR targets main so GitHub CI runs; the navigation change itself is the subsequent five-file commit.

Validation: source review, changed-range formatting and git diff --check. Local builds, tests, revert checks and CI were explicitly waived for this campaign continuation; runtime validation is pending on GitHub.

Refs #1250.

Refs #2291, #706, #1147. Preserve raw argument values and carry URL projections separately through compaction and spill.

Signed-off-by: Martin Vogel <martin.vogel.tech@gmail.com>
Signed-off-by: Martin Vogel <martin.vogel.tech@gmail.com>
Refs #1250

Signed-off-by: Martin Vogel <martin.vogel.tech@gmail.com>
@DeusData

DeusData commented Oct 3, 2026

Copy link
Copy Markdown
Owner Author

Checkpoint and handover (2026-10-03 UTC)

Published head: 49fc531758def04d54c6fe4bf76a7cca6097b407. The remote head was verified.

The client-navigation fix for #1250 depends on URL/topic folding in #2513. Review and validate in that dependency order. Local runtime verification remains unrun under the waiver.

Hosted snapshot at 2026-10-03 21:54:18 UTC: 18 queued, 4 skipped, 7 success. Confirm the required checks on this exact head before treating it as ready.

No additional local build, test, lint, sanitizer, benchmark or CI runs were performed at this checkpoint, as requested. Earlier executed evidence remains historical; prepared tests and the newer source-reviewed changes must still be validated by the hosted gate.

The campaign is paused at the maintainer’s request. Local monitoring has stopped; hosted jobs remain running. No merge was performed. Thanks for reviewing this change.

This branch has not been deployed

No deployments
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