docs(oschema): drop manual sid/created_at/updated_at — generator auto-injects #278

Merged

timur merged 1 commit from issue-275-oschema-skill-sid into development

2026-05-20 23:31:51 +00:00

timur commented

2026-05-20 23:31:44 +00:00

Owner

Summary

Rephrase the Root Objects rule: [rootobject] is sufficient; authors must NOT declare sid, created_at, or updated_at.
Add an auto-injection callout naming the real generated types (sid: SmartId, created_at: u64, updated_at: u64).
Scrub every example block (Task, Product, Project, SID demos, naming-convention examples) of manually declared auto-fields.
Convert reference fields from project_sid: str to the typed project_sid: sid.

Test plan

grep 'sid: str' skills/oschema/oschema.md → 0 hits.
grep 'created_at\|updated_at' skills/oschema/oschema.md → only the new auto-injection callout.

#275

## Summary - Rephrase the Root Objects rule: `[rootobject]` is sufficient; authors must NOT declare `sid`, `created_at`, or `updated_at`. - Add an auto-injection callout naming the real generated types (`sid: SmartId`, `created_at: u64`, `updated_at: u64`). - Scrub every example block (Task, Product, Project, SID demos, naming-convention examples) of manually declared auto-fields. - Convert reference fields from `project_sid: str` to the typed `project_sid: sid`. ## Test plan - `grep 'sid: str' skills/oschema/oschema.md` → 0 hits. - `grep 'created_at\|updated_at' skills/oschema/oschema.md` → only the new auto-injection callout. https://forge.ourworld.tf/lhumina_code/hero_skills/issues/275

timur added 1 commit

2026-05-20 23:31:44 +00:00

oschema skill: remove manual sid/created_at/updated_at declarations 8118f8515e

The hero_rpc generator (rust_struct.rs) auto-injects `sid: SmartId`,
`created_at: u64`, `updated_at: u64` on every `[rootobject]` and silently
drops any schema-declared duplicates, so the skill's "MUST have sid: str"
rule was wrong and the examples were misleading.

- Rephrase the root-objects rule: `[rootobject]` is sufficient; sid,
  created_at, updated_at must NOT be declared in schema.
- Add an explicit callout about auto-injection and the actual generated
  types (`SmartId`, `u64`).
- Drop `sid: str` / `created_at: otime` / `updated_at: otime` from every
  example block (Task, Product, Project, User-SmartID demos, Name-Fields
  consistency examples).
- Convert reference fields like `project_sid: str` to the typed `sid` form
  so zero `sid: str` hits remain in example blocks.

Refs: #275

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

timur merged commit 0dfbe09d3b into development

2026-05-20 23:31:51 +00:00

timur deleted branch issue-275-oschema-skill-sid

2026-05-20 23:31:51 +00:00

timur referenced this pull request from a commit

2026-05-20 23:31:52 +00:00

docs(oschema): drop manual sid/created_at/updated_at — generator auto-injects (#278)

timur referenced this pull request

2026-05-20 23:32:00 +00:00

oschema skill prescribes manually declaring sid/created_at/updated_at — generator already auto-injects them #275