Skip to content

Index nested field topology and preserve generated metadata - #3

Merged
trissim merged 2 commits into
mainfrom
agent/index-nested-field-topology
Jul 30, 2026
Merged

Index nested field topology and preserve generated metadata#3
trissim merged 2 commits into
mainfrom
agent/index-nested-field-topology

Conversation

@trissim

@trissim trissim commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

What changed

  • index immutable direct-child field paths when canonical flat state is extracted or restored
  • expose direct-child and descendant queries from ObjectState
  • centralize structural replacement and remove invalid partial-state fallbacks
  • preserve declaration docs, module provenance, same-name qualnames, and runtime first-line metadata across every dataclass recreation boundary
  • preserve inherited annotated validation through lazy and injected configuration types
  • pin quality tools and make Ruff block fatal package errors plus diagnostics on Git-derived added lines, without a synchronized violation baseline
  • report the historical Black and Mypy baselines explicitly as non-blocking until they are migrated
  • release the change as 1.0.21

Why

ObjectState owns both flattened field topology and synthesized lazy/global configuration classes. Consumers should query its derived topology rather than repeatedly rescanning values, and generated classes should retain the documentation and validation semantics of their declaration owners.

Validation

  • full ObjectState suite: 158 passed
  • real OpenHCS global/pipeline/lazy config smoke: declaration docs preserved and invalid annotated values rejected
  • full-package Ruff fatal gate: passed
  • Git-derived added-line Ruff gate over 10 changed Python files: passed
  • source and workflow diff checks: passed

@trissim trissim changed the title Index nested ObjectState field topology Index nested field topology and preserve generated metadata Jul 30, 2026
@trissim
trissim marked this pull request as ready for review July 30, 2026 17:47
Copilot AI review requested due to automatic review settings July 30, 2026 17:47
@trissim
trissim merged commit 4e0665f into main Jul 30, 2026
11 checks passed
@trissim
trissim deleted the agent/index-nested-field-topology branch July 30, 2026 17:48

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@augmentcode

augmentcode Bot commented Jul 30, 2026

Copy link
Copy Markdown

PR Risk Analyzer Agent🛡️

👀 Human Input Needed
A pair-review briefing is ready for you.

→ Pair Review Briefing

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