Skip to content

feat(integrations): add laravel-cloud Stripe catalog provider#42

Open
snowmead wants to merge 1 commit into
mainfrom
feat/integration-laravel-cloud
Open

feat(integrations): add laravel-cloud Stripe catalog provider#42
snowmead wants to merge 1 commit into
mainfrom
feat/integration-laravel-cloud

Conversation

@snowmead

@snowmead snowmead commented Jul 11, 2026

Copy link
Copy Markdown
Owner

Summary

  • Adds the laravel-cloud Stripe Projects catalog integration family as CatalogResource adapter(s).
  • Follows docs/ADDING-A-PROVIDER.md and docs/PROVIDER-WAVES.md.
  • Includes gap test + hermetic provision test. OUTPUT_FIELDS provisional until mise run discover.

Test plan

  • Gap + hermetic tests in module
  • Live mise run discover when provider is linked
  • Merge after resolving additive registry.rs conflict with other family PRs

Note

Low Risk
Additive catalog integrations with offline tests and registry-only dispatch changes; provisional output envelopes are the main follow-up before live provision.

Overview
Introduces Laravel Cloud as a Stripe Projects catalog family with three managed CatalogResource adapters: laravel_cloud/application (laravel-cloud, outputs app_id), laravel_cloud/mysql (laravel-cloud-mysql, database_url), and laravel_cloud/valkey (laravel-cloud-valkey, redis_url). Each module follows the standard pattern—catalog-aligned config, validate_config, gap tests, and hermetic provision_script tests—with provisional OUTPUT_FIELDS until mise run discover.

Wires the family into register_providers!, providers/mod.rs, and extends the Hostable::OUTPUTS vs OUTPUT_FIELDS drift test for all three types.

stackless-provider-sdk gains int_optional, bool_required, and bool_optional for generated/catalog schemas that use non-string fields.

Documentation adds PROVIDER-WAVES.md (parallel Phase 1 landing, merge gates, exclusions), a catalog-first decision in DECISIONS.md, and a pointer from ADDING-A-PROVIDER.md. scripts/generate_catalog_integrations.py scaffolds future deployable adapters from catalog.json (Laravel Cloud entries included in its hints map).

This PR does not add a --on laravel cloud substrate or live smoke fixtures.

Reviewed by Cursor Bugbot for commit 1cb7934. Bugbot is set up for automated code reviews on this repo. Configure here.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 1cb7934. Configure here.

"vercel/project",
"flyio/app",
"netlify/project",
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Generator skips implemented Laravel

Medium Severity

The same commit adds hand-written laravel_cloud adapters and a catalog generator whose IMPLEMENTED set still omits laravel_cloud/application, mysql, and valkey. Those refs appear in SHORT_PROVIDER and OUTPUT_HINTS but not in IMPLEMENTED, so running the script treats them as unimplemented and can rewrite the family and append duplicate register_providers! rows.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 1cb7934. Configure here.

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