Skip to content

feat: rename storage_table_provisions → provisions, remove storage_config.policies type#1039

Merged
pyramation merged 2 commits intomainfrom
feat/consolidate-storage-provisions
Apr 30, 2026
Merged

feat: rename storage_table_provisions → provisions, remove storage_config.policies type#1039
pyramation merged 2 commits intomainfrom
feat/consolidate-storage-provisions

Conversation

@pyramation
Copy link
Copy Markdown
Contributor

Summary

Updates TypeScript blueprint types to match the storage policy consolidation in constructive-db#962.

Changes

  1. Remove BlueprintStoragePolicy type — no longer needed since storage_config.policies was removed
  2. Remove policies field from BlueprintStorageConfig interface
  3. Rename storage_table_provisionsprovisions in BlueprintStorageConfig
  4. Update JSDoc to reflect provisions consolidation
  5. Regenerate blueprint-types.generated.ts

Companion PRs

  • constructive-io/constructive-db#962 — SQL-level consolidation
  • constructive-io/constructive-skills — Skill docs update

Review & Testing Checklist for Human

Risk: 🟢 green (type-only changes, no runtime behavior)

  • Verify the generated BlueprintStorageConfig type no longer has policies or storage_table_provisions
  • Confirm provisions key has the correct shape ({ files?, buckets?, upload_requests? })

Notes

  • BlueprintStoragePolicy was only referenced by the removed policies field, so removing it is safe

Link to Devin session: https://app.devin.ai/sessions/ff0b16d790a34cceb17740c9897a295e
Requested by: @pyramation

…onfig.policies

- Remove BlueprintStoragePolicy type (no longer needed)
- Remove policies field from BlueprintStorageConfig
- Rename storage_table_provisions -> provisions in BlueprintStorageConfig
- Update JSDoc to reflect provisions consolidation
- Regenerate blueprint-types.generated.ts
@devin-ai-integration
Copy link
Copy Markdown
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@pyramation pyramation merged commit 4c56615 into main Apr 30, 2026
52 checks passed
@pyramation pyramation deleted the feat/consolidate-storage-provisions branch April 30, 2026 03:27
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.

1 participant