chore: regenerate models from upstream schemas #103

Merged
stack72 merged 1 commit from automated/regenerate-models into main 2026-07-14 19:55:30 +00:00
Owner

Summary

Automated regeneration of extension models from upstream provider schemas.

Schema Sources

  • AWS: CloudFormation Resource Schema
  • GCP: Google Discovery Documents
  • Cloudflare: Cloudflare OpenAPI spec
  • Hetzner: Hetzner Cloud OpenAPI spec
  • DigitalOcean: DigitalOcean OpenAPI spec

Review Notes

  • Files under model/ are auto-generated — review the manifest.yaml diffs for version changes
  • CalVer versioning with content-based change detection ensures versions only bump when content changes
  • Publishing happens automatically when this PR is merged (via the publish workflow)
## Summary Automated regeneration of extension models from upstream provider schemas. ### Schema Sources - **AWS**: CloudFormation Resource Schema - **GCP**: Google Discovery Documents - **Cloudflare**: Cloudflare OpenAPI spec - **Hetzner**: Hetzner Cloud OpenAPI spec - **DigitalOcean**: DigitalOcean OpenAPI spec ### Review Notes - Files under `model/` are auto-generated — review the `manifest.yaml` diffs for version changes - CalVer versioning with content-based change detection ensures versions only bump when content changes - Publishing happens automatically when this PR is merged (via the publish workflow)
chore: regenerate models from upstream schemas (2026-07-14)
All checks were successful
CI / cve/mini-shai-hulud - lint (pull_request) Has been skipped
CI / cve/mini-shai-hulud - fmt (pull_request) Has been skipped
CI / cve/mini-shai-hulud - test (pull_request) Has been skipped
CI / cve/researcher - check (pull_request) Has been skipped
CI / cve/researcher - lint (pull_request) Has been skipped
CI / cve/researcher - fmt (pull_request) Has been skipped
CI / cve/researcher - test (pull_request) Has been skipped
CI / cve/dirtyfrag - lockfile up to date (pull_request) Has been skipped
CI / cve/mini-shai-hulud - lockfile up to date (pull_request) Has been skipped
CI / Dependency Audit (pull_request) Successful in 3m49s
CI / cve/researcher - lockfile up to date (pull_request) Has been skipped
CI / software-factory - check (pull_request) Has been skipped
CI / software-factory - lockfile up to date (pull_request) Has been skipped
CI / software-factory - fmt (pull_request) Has been skipped
CI / software-factory - test (pull_request) Has been skipped
CI / software-factory - lint (pull_request) Has been skipped
CI / codegen - check (pull_request) Has been skipped
CI / codegen - fmt (pull_request) Has been skipped
CI / codegen - lint (pull_request) Has been skipped
CI / CI Security Review (pull_request) Has been skipped
CI / codegen - lockfile up to date (pull_request) Has been skipped
CI / model/digitalocean - check (pull_request) Successful in 58s
CI / cloudflare models - lockfiles up to date (pull_request) Successful in 1m6s
CI / cloudflare models - sample check (pull_request) Successful in 1m7s
CI / aws models - lockfiles up to date (pull_request) Successful in 1m20s
CI / gcp models - lockfiles up to date (pull_request) Successful in 1m12s
CI / gcp models - sample check (pull_request) Successful in 1m34s
CI / Adversarial Code Review (pull_request) Has been skipped
CI / Claude Code Review (pull_request) Successful in 3m9s
CI / Merge Gate (pull_request) Successful in 27s
20dfa226df
Author
Owner

Code Review

Blocking Issues

None.

Suggestions

  1. Review instructions gap — nightly regeneration isn't explicitly listed as a legitimate case for model-only changes.
    The CLAUDE.md review rules list two cases where model/ may change without codegen/ changes: (a) codegen regeneration where codegen/ also changes, and (b) version-only bumps via bump-versions. This PR falls into neither category by strict reading — it has new files and new schema fields, not just version/upgrade entries, and no codegen/ files changed.

    However, all evidence points to this being a legitimate output of the nightly regenerate-models CI workflow (recently expanded to include Cloudflare in PR #102):

    • Every new and modified file carries the // Auto-generated … Do not edit manually header.
    • The single commit is titled chore: regenerate models from upstream schemas (2026-07-14), matching the nightly workflow's commit convention.
    • Content patterns (schema structure, credential handling, version CalVer dating) are mechanically consistent across all 84 files — not hand-authoring at scale.
    • New packages (data-security, one, registrar-sandbox) include all required scaffolding: deno.json, deno.lock, manifest.yaml, LICENSE.txt, README.md.
    • deno.json imports and deno.lock pin zod to an exact version (4.3.6), consistent with the project rule.

    Recommendation: Update the review instructions in CLAUDE.md (or a separate review guide) to explicitly list nightly schema regeneration — where only model/ changes but the codegen pipeline code is unchanged — as a third legitimate case, so future automated reviews don't treat this workflow as suspect.

## Code Review ### Blocking Issues None. ### Suggestions 1. **Review instructions gap — nightly regeneration isn't explicitly listed as a legitimate case for model-only changes.** The `CLAUDE.md` review rules list two cases where `model/` may change without `codegen/` changes: (a) codegen regeneration where `codegen/` also changes, and (b) version-only bumps via `bump-versions`. This PR falls into neither category by strict reading — it has new files and new schema fields, not just version/upgrade entries, and no `codegen/` files changed. However, all evidence points to this being a legitimate output of the nightly `regenerate-models` CI workflow (recently expanded to include Cloudflare in PR #102): - Every new and modified file carries the `// Auto-generated … Do not edit manually` header. - The single commit is titled `chore: regenerate models from upstream schemas (2026-07-14)`, matching the nightly workflow's commit convention. - Content patterns (schema structure, credential handling, version CalVer dating) are mechanically consistent across all 84 files — not hand-authoring at scale. - New packages (`data-security`, `one`, `registrar-sandbox`) include all required scaffolding: `deno.json`, `deno.lock`, `manifest.yaml`, `LICENSE.txt`, `README.md`. - `deno.json` imports and `deno.lock` pin `zod` to an exact version (`4.3.6`), consistent with the project rule. **Recommendation:** Update the review instructions in `CLAUDE.md` (or a separate review guide) to explicitly list nightly schema regeneration — where only `model/` changes but the codegen pipeline code is unchanged — as a third legitimate case, so future automated reviews don't treat this workflow as suspect.
stack72 deleted branch automated/regenerate-models 2026-07-14 19:55:31 +00:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
swamp-club/swamp-extensions!103
No description provided.