FIDL Project Checklist
Changes to FIDL can often have a large number of downstream effects. This document provides checklists for items that should be considered when changing FIDL. Note that not all elements of the checklist will necessarily apply to every change, but they should at least be helpful for developers to double check that their changes are complete.
Change to the fidlc
compiler
This checklist is for changes that affect FIDL itself, such as syntax or semantics changes.
- Update specifications.
- Update associated tools.
- The FIDL formatter/linter and their associated TreeVisitors.
- The fidldoc tool, for language changes that should be reflected in the generated documentation, e.g. a new data type or modifier.
- Editors and syntax highlighting.
- Update FIDL files. For syntax changes, update existing FIDL source including:
- Ensure that interactions with attributes are handled.
- For example, does the feature require a new placement?
- Update integration tests
- For larger language features, it may be worth adding test coverage in existing integration tests. For example, ensuring cross binding consistency through the compatibility tests, or transitionability through the source compatibility tests.
- For parser changes specifically:
- Update the
span_tests
to ensure that the parsed spans are correct, and not just that parsing succeeds
- For raw AST changes:
- Ensure updates to the AST are propagated to the raw AST tree visitor and its downstream users as well. For example, adding a new field to a node probably should be accompanied with a visitor change that visits that new field.
- For semantics changes:
- Update the feature specific unittest, e.g.
table_tests
when making a modification to tables, or create a new test file.
It is expected for changes to FIDL which cascade to backends to update all Fuchsia FIDL team owned bindings, i.e. Rust, Go, Dart, HLCPP, Unified C++ (was LLCPP).
Change to the JSON IR
- Esnure that the change is reflected by a change in the IR goldens.
- Update the JSON schema
- Update any JSON consumers, including:
Change to fidlgen_<lang>
- Update the bindings docs if the generated code has changed.
- Update the bindings tutorials to reflect the current best practices and patterns.
- Update any relevant [guides].
- FIDL API rubric for guidance on using this feature.
- For example, when changing the memory allocation APIs in LLCPP, the LLCPP allocator guide should be updated.
- We've also found it good practice to present to API Council in order to socialize new features, and explain to council members how to review APIs in light of evolutions.
Horizontal testing requirements
Add coverage to: