Skip to content

Point sync workflow at TemplateMarket edge function#46

Merged
CarmenDou merged 1 commit into
mainfrom
feat/sync-url-to-templatemarket
May 27, 2026
Merged

Point sync workflow at TemplateMarket edge function#46
CarmenDou merged 1 commit into
mainfrom
feat/sync-url-to-templatemarket

Conversation

@CarmenDou
Copy link
Copy Markdown
Contributor

@CarmenDou CarmenDou commented May 27, 2026

Summary

  • Re-points the registry sync workflow from api.insforge.dev/admin/v1/templates/sync (cloud-backend Express controller) to p8n7m7ci.us-east.insforge.app/functions/sync-templates (TemplateMarket InsForge project's edge function).
  • The TemplateMarket project (c6366c10-…) now owns the templates table, RPCs, and sync logic; cloud-backend will drop its template.controller.ts + 073_template_marketplace.sql migration in a follow-up.
  • Repo TEMPLATE_SYNC_TOKEN secret has been rotated to match the value stored on the TemplateMarket project — manual sync smoke-tested 200 with synced=7.

Test plan

  • Workflow run on merge to main (the sync path filters apply) returns HTTP 200
  • templates table row count on TemplateMarket project matches registry.json length
  • Once cloud-backend drops its endpoint, no other workflow / consumer still hits api.insforge.dev/admin/v1/templates/sync

Summary by cubic

Points the registry sync workflow to TemplateMarket’s edge function at https://p8n7m7ci.us-east.insforge.app/functions/sync-templates, replacing https://api.insforge.dev/admin/v1/templates/sync. Sync now targets the TemplateMarket project; the repo TEMPLATE_SYNC_TOKEN was rotated accordingly.

Written for commit 1c55733. Summary will update on new commits. Review in cubic

Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

Re-trigger cubic

Copy link
Copy Markdown
Member

@jwfing jwfing left a comment

Choose a reason for hiding this comment

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

LGTM, approved.

@CarmenDou CarmenDou merged commit af6a860 into main May 27, 2026
4 checks passed
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.

2 participants