Skip to content

Complete and navigate Tempest view data arguments - #561

Open
shuvroroy wants to merge 2 commits into
PHPantom-dev:mainfrom
shuvroroy:feat/tempest-view-arguments
Open

shuvroroy wants to merge 2 commits into
PHPantom-dev:mainfrom
shuvroroy:feat/tempest-view-arguments

Conversation

@shuvroroy

Copy link
Copy Markdown
Contributor

Named data passed to Tempest's view() now completes from the receiving template's inputs. A label such as books: jumps to the template's first read of $books and shows its normal PHP type in a View parameter hover.

The lookup reverses the path resolution already used for view-data inference and reads only the selected template. It excludes data already supplied by the call, local bindings, and isolated scopes, while preserving closure captures, conditional bindings, unsaved edits, and template source coordinates. Custom View constructors keep their ordinary parameter handling.

This is the separate follow-up to #560; its branch starts from main and contains only view-argument changes.

Validation:

  • Full cargo test suite passes.
  • cargo clippy --fix --allow-dirty -- -D warnings, then cargo fmt, and cargo clippy --all-targets -- -D warnings pass.
  • LLVM coverage verifies 100% of changed executable Rust lines.
  • All contributing-guide PHP lint and runtime assertion checks pass. Framework analysis is clean, with exactly the four intentional Laravel errors documented by the guide.

Checklist:

  • Updated the changelog and documentation.
  • Added regression tests and working playground examples.
  • Removed the completed roadmap item from both listings and its domain document.
  • Reviewed the generated code and user-facing text.

Resolve the template through the existing view-data path lookup and offer
its input names instead of the variadic params container. Navigate labels
to their first source reads and show the template's normal PHP hover type.

Read only the selected template, using indexed variable bindings and source
maps to distinguish incoming data from locals and isolated scopes. Cover
spreads, imports, captures, conditional bindings, incomplete calls, unsaved
edits, stale metadata, and calls made from templates.
Add a title input to the view playground and verify named data reaches
both populated and empty views with Tempest's renderer. Document completion,
navigation, and hover, credit the contribution, and remove the completed
roadmap item while preserving release and sprint markers.
@codecov-commenter

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@shuvroroy
shuvroroy marked this pull request as ready for review November 9, 2026 05:30
@shuvroroy shuvroroy mentioned this pull request Nov 9, 2026

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.

2 participants