fix(angular): fix 6 compiler divergences from Angular reference#31
Merged
Conversation
1. viewport({trigger: ...}) now keeps empty options object (was dropping it)
2. Timer delays preserve fractional precision (u32 -> f64, e.g. 1.5ms)
3. Pipe detection in @for track now traverses all AST expression types
4. hover/interaction triggers enforce max 1 parameter; never removed from on-triggers
5. @for let assignment a=b=c correctly rejected (split instead of splitn)
6. @for expression parse failure no longer emits extra missing-track error
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com
Note
Medium Risk
Touches template parsing and IR/reify generation for
@deferand@for, so behavior changes may affect emitted output and error reporting across many templates despite being targeted compatibility fixes.Overview
Aligns
@defertrigger handling with Angular by preserving fractional timer delays end-to-end (u32->f64), keeping empty viewport options objects forviewport({trigger: ...}), and tightening trigger validation (enforce max 1 parameter forhover/interaction, and rejectneveras anontrigger).Improves
@forparsing/validation to match Angular: adds anexpression_parse_failedflag to avoid emitting an extra missing-trackerror on expression parse failures, rejects malformedletassignments likea=b=c, and expands pipe detection in track expressions to traverse more AST node types. Adds new e2e fixtures covering defer edge cases (empty viewport options + fractional timer durations).Written by Cursor Bugbot for commit 574d59f. This will update automatically on new commits. Configure here.