chore(agentic-ci): add future annotations import to dataset_metadata.py#640
Conversation
Per AGENTS.md: every Python source file requires `from __future__ import annotations`.
Greptile SummaryThis PR adds the missing
|
| Filename | Overview |
|---|---|
| packages/data-designer-config/src/data_designer/config/dataset_metadata.py | Adds from __future__ import annotations after the SPDX header block to comply with the repo-wide structural invariant; no functional change. |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart TD
A["dataset_metadata.py"] --> B["SPDX Header Comments"]
B --> C["from __future__ import annotations ✅ (added)"]
C --> D["from pydantic import BaseModel"]
D --> E["class DatasetMetadata(BaseModel)"]
E --> F["seed_column_names: list[str] = []"]
Reviews (3): Last reviewed commit: "Merge branch 'main' into agentic-ci/chor..." | Re-trigger Greptile
eric-tramel
left a comment
There was a problem hiding this comment.
Incredible find agentic-ci 🫡
|
Thank you for your submission! We ask that you all sign our Developer Certificate of Origin before we can accept your contribution. You can sign the DCO by adding a comment below using this text: I have read the DCO document and I hereby sign the DCO. 1 out of 2 committers have signed the DCO. |
|
/authorize-agentic-ci |
Summary
Add
from __future__ import annotationstopackages/data-designer-config/src/data_designer/config/dataset_metadata.py.Why
Per
AGENTS.md("Structural Invariants"):The structural-integrity audit detected this file as the only source file
under
packages/*/src/missing the import.What changed
A single line —
from __future__ import annotations— inserted after theSPDX header block and before existing imports. No functional change.
Tests
make test-config— 562 passed.Provenance
Generated by the agentic-ci
structuresuite (daily run, 2026-05-13).Finding id
dc0ea67d384e, fix confidence 1.0.