Commit 9356d54
committed
Fix rule_id attribution, $data coercion, Fn/ dispatch, and rule bugs
Engine fixes:
1. KeywordRuleRegistry::dispatch stamps rule_id on errors from rules
2. Walker inline validation respects existing rule_id on errors
3. json_schema_rule sets rule_id on all returned errors
4. Numeric validators coerce string constraints for $data resolution
5. Walker dispatches Fn/{name} synthetic keywords for Function nodes
6. Walker inline validation respects pre-set rule_id
Rule fixes:
7. W2506: extract param name from Ref node (not pre-resolved string)
8. E3675: use validate_regional (schema is region-keyed, not flat)
All 10 previously-silent rules now fire correctly:
E3028, E3009, E6102, E4001, E4002, E3706, E3712, W1019, W2506, E3675
133/133 integration tests pass. Zero known gaps.1 parent 8b73136 commit 9356d54
49 files changed
Lines changed: 87248 additions & 36460 deletions
File tree
- crates/cfn-lint
- data/schemas/extensions
- aws_amazonmq_broker
- aws_appstream_fleet
- aws_docdb_dbinstance
- aws_dynamodb_table
- aws_ec2_instance
- aws_ec2_vpngateway
- aws_ecs_taskdefinition
- aws_elasticache_cachecluster
- aws_elasticsearch_domain
- aws_emr_cluster
- aws_gamelift_fleet
- aws_lambda_function
- aws_lambda_permission
- aws_neptune_dbinstance
- aws_rds_dbcluster
- aws_rds_dbinstance
- aws_redshift_cluster
- aws_sagemaker_cluster
- aws_sagemaker_hosting
- src
- jsonschema
- keywords
- rules
- parameters
- resources/other
- walker
- tests/integration/rules
- metadata
- outputs
- parameters
- resources
- templates
Some content is hidden
Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.
Lines changed: 24 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
5 | 5 | | |
6 | 6 | | |
7 | 7 | | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
8 | 16 | | |
9 | 17 | | |
10 | 18 | | |
| |||
222 | 230 | | |
223 | 231 | | |
224 | 232 | | |
| 233 | + | |
| 234 | + | |
| 235 | + | |
| 236 | + | |
| 237 | + | |
| 238 | + | |
| 239 | + | |
| 240 | + | |
| 241 | + | |
| 242 | + | |
| 243 | + | |
| 244 | + | |
| 245 | + | |
| 246 | + | |
| 247 | + | |
| 248 | + | |
225 | 249 | | |
226 | 250 | | |
227 | 251 | | |
| |||
0 commit comments