stop deprecated Extra.allow access in Node

[xyf5432]

Fixes #87

Summary

Replaces the deprecated Extra.allow member access inside Node's nested
class Meta in src/constants/kg.py with the literal string "allow", and drops
the now-unused Extra import.

Every import of src/constants/kg.py under pydantic v2 currently emits a
PydanticDeprecatedSince20 warning. This removes it.

Changes

• src/constants/kg.py: extra = Extra.allow → extra = "allow"; import line
  drops Extra (BaseModel, ConfigDict, Field).

Notes

• The nested class Meta is inert as far as pydantic is concerned (only
  Config/model_config are honored), so this is a strict no-behavior-change fix.
  If the docstring intent — actually allowing extra properties — is wanted, that is
  a separate change (model_config = ConfigDict(extra="allow") on Node) that
  should land deliberately, as it changes runtime behavior.

Testing

• Verified with pydantic 2.13.4: importing the module emits zero warnings
  (before: 1 DeprecationWarning), model construction behavior unchanged.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 5976f3b5-9079-49d5-9840-59b8ec151275

📥 Commits

Reviewing files that changed from the base of the PR and between 61dc3ec and d4ad0b5.

📒 Files selected for processing (1)

• src/constants/kg.py

Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review.

---

📝 Walkthrough

Walkthrough

The change updates the Pydantic import in src/constants/kg.py and changes Node.Meta.extra from Extra.allow to "allow".

Changes

Pydantic configuration update

Layer / File(s)	Summary
Node model configuration src/constants/kg.py	The module replaces the Extra import with Field. Node.Meta.extra now uses the string value "allow".

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to d4ad0

This localized change removes a deprecation warning without changing Node validation or caller-visible behavior. No actionable merge-blocking risk remains after normal checks and review.

Poem

A rabbit checks the model’s gate
"allow" now marks the state
Extra hops away
Field joins the array
Small changes keep the schema straight

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and concisely describes the main change: removing deprecated Extra.allow access from Node.

Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1 files.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}