feat: add AlignedDynamicTable category validation utilities

[ehennestad]

Accidentally merged to wrong branch, see PR #837 instead

[ehennestad]

Fixes bug where column height calculation ignored a non-zero offset for a DataPipe. Probably rare, non-zero offset is supported by api and should be accounted for.

[codecov]

Codecov Report

❌ Patch coverage is 90.94203% with 25 lines in your changes missing coverage. Please review.
✅ Project coverage is 95.13%. Comparing base (ad0fcb0) to head (60d8028).
⚠️ Report is 29 commits behind head on codex/report-schema-violation-context.

Files with missing lines	Patch %	Lines
+matnwb/+neurodata/AlignedDynamicTableBase.m	85.89%	22 Missing ⚠️
...es/+util/+dynamictable/+internal/getColumnHeight.m	90.00%	2 Missing ⚠️
...nternal/collectConstantPropertiesAcrossHierarchy.m	91.66%	1 Missing ⚠️

Additional details and impacted files

@@                            Coverage Diff                            @@
##           codex/report-schema-violation-context     #833      +/-   ##
=========================================================================
- Coverage                                  95.38%   95.13%   -0.26%     
=========================================================================
  Files                                        221      223       +2     
  Lines                                       7894     7913      +19     
=========================================================================
- Hits                                        7530     7528       -2     
- Misses                                       364      385      +21     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[Copilot]

Pull request overview

This PR introduces first-class category utilities for AlignedDynamicTable by adding a shared non-generated base class that implements addCategory/getCategory, schema-category routing, and unified runtime validation to enforce category/row-height alignment. It also updates the code generator to emit schema-category discovery hooks and updates the icephys tutorial + unit tests to reflect the new API and stricter alignment validation.

Changes:

• Add matnwb.neurodata.AlignedDynamicTableBase and integrate it into generated AlignedDynamicTable classes for category registration/lookup and alignment validation.
• Improve DynamicTable height/id inference utilities (getTableHeight, getColumnHeight, initDynamicTableId) and add unit tests.
• Update generator logic (constructor/custom-constraint/property hooks) to support schema-defined category discovery and auto-registration; update the icephys tutorial accordingly.

Reviewed changes

Copilot reviewed 19 out of 21 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
tutorials/private/mcode/icephys.m	Updates tutorial usage to avoid pre-listing categories with populated tables and uses addCategory for custom categories.
+types/+util/+dynamictable/+internal/initDynamicTableId.m	Refactors id initialization to handle existing ids/DataPipes more carefully.
+types/+util/+dynamictable/+internal/getTableHeight.m	Returns both inferred height and whether height is “established”.
+types/+util/+dynamictable/+internal/getColumnHeight.m	Adjusts unbound DataPipe height inference to include offset + queued data.
+types/+hdmf_common/AlignedDynamicTable.m	Mixes in AlignedDynamicTableBase and switches validation to ensureAlignedTableConsistency.
+types/+core/IntracellularRecordingsTable.m	Uses new alignment consistency validation and auto-registers schema category names on assignment.
+tests/+unit/dynamicTableTest.m	Adds tests for new height semantics and id initialization behavior.
+tests/+unit/alignedDynamicTableTest.m	Adds comprehensive tests for addCategory/getCategory and alignment validation.
+tests/+doubles/SchemaCategoryAlignedTable.m	Test double for a schema-defined category property.
+tests/+doubles/ElementIdentifiersStub.m	Minimal ElementIdentifiers test double.
+tests/+doubles/DynamicTableStub.m	Minimal DynamicTable test double.
+tests/+doubles/AlignedDynamicTableStub.m	Minimal AlignedDynamicTable test double using the new base class.
+matnwb/+neurodata/AlignedDynamicTableBase.m	New core implementation of category utilities + alignment validation.
+file/isDynamicTableDescendant.m	Removes old descendant check helper.
+file/getPropertyHooks.m	Adds generator hooks for AlignedDynamicTable.categories validation + category postset registration.
+file/fillCustomConstraint.m	Generates checkCustomConstraint for AlignedDynamicTable to enforce consistency.
+file/fillConstructor.m	Generates constructor-time consistency checks for AlignedDynamicTable descendants.
+file/fillClass.m	Adds AlignedDynamicTableBase as a superclass and generates schema-category discovery method blocks.
+file/+internal/isDescendantOf.m	New generalized “is descendant of” helper for generator logic.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[ehennestad]

Accidentally merged via merge queue when base was changed from main to another branch because stacking was needed