Skip to content

docs(openspec): tick verified cloudflare-migration tasks#536

Merged
pedropaulovc merged 1 commit into
mainfrom
chore/cloudflare-migration-task-status
Jun 2, 2026
Merged

docs(openspec): tick verified cloudflare-migration tasks#536
pedropaulovc merged 1 commit into
mainfrom
chore/cloudflare-migration-task-status

Conversation

@pedropaulovc
Copy link
Copy Markdown
Collaborator

@pedropaulovc pedropaulovc commented Jun 2, 2026

Corrects the migrate-deploy-to-cloudflare task checkboxes to match reality — doc-only, no code.

Ticked (with evidence):

  • 2.1 Workers Builds connected & deploying — required Workers Builds: codjiflo check + live codjiflo.net
  • 2.4 no .env.local needed locally (gh token / dev auto-login, PR feat(auth): use GitHub CLI token for local dev + prod-mode E2E #535)
  • 3.6 GitHub App homepage + OAuth callback URLs → codjiflo.net (done in dashboard)
  • 6.3 codjiflo.net/api/health serves current main HEAD; e2e-tests-prod green

Still open (external/manual, intentionally left unchecked):

  • 2.6 custom *.codjiflo.net preview domain (workers.dev fallback in use)
  • 5.3 pause Vercel + disconnect its GitHub App — still deploying in parallel
  • 5.4 codjiflo.vza.netcodjiflo.net 301 (needs the vza.net Cloudflare account)

Not archiving the change yet — these 3 remain.


🔍 Review in CodjiFlo

Mark tasks now confirmed done, with evidence:
- 2.1 Cloudflare Workers Builds connected & deploying (required check + live site)
- 2.4 no .env.local needed locally (gh token / dev auto-login; PR #535)
- 3.6 GitHub App homepage + OAuth callback URLs → codjiflo.net (dashboard)
- 6.3 codjiflo.net/api/health serves current main HEAD; e2e-tests-prod green

Still open (external/manual): 2.6 custom *.codjiflo.net preview domain,
5.3 pause Vercel + disconnect its GitHub App (still deploying in parallel),
5.4 codjiflo.vza.net → codjiflo.net 301 redirect (needs the vza.net account).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings June 2, 2026 03:33
@pedropaulovc pedropaulovc enabled auto-merge June 2, 2026 03:33
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 2, 2026

CodjiFlo Iteration Tracking

Iterations captured: 1
Last updated: 2026-06-02T03:33:59.000Z
Artifact: 7348402604
Run ID: 26796667971


What is this?

This comment is automatically updated by the CodjiFlo GitHub Action to enable force-push resilient code review with iteration tracking.

The artifact referenced above contains iteration data that the CodjiFlo frontend uses to:

  • Track code changes across force-pushes
  • Enable comment persistence across code modifications
  • Allow comparison between any two iterations

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the Cloudflare migration task checklist in OpenSpec to reflect the current verified deployment/migration status (documentation-only).

Changes:

  • Marked task 2.1 as complete and expanded the verification notes for Workers Builds + live production.
  • Marked task 2.4 as complete and documented that local dev no longer requires .env.local (referencing PR #535).
  • Marked task 3.6 and 6.3 as complete with updated verification notes (GitHub App URLs + live health/E2E-prod).

## 2. Cloudflare Git release process & Secret Store (via `pedrovezzadev`)

- [ ] 2.1 Sign in as `pedrovezzadev`; connect the GitHub repo in Cloudflare (Workers Builds), production branch = `main`
- [x] 2.1 Cloudflare Workers Builds is connected and deploying: the `Workers Builds: codjiflo` check runs (and is required) on every PR, and production on `main` serves `codjiflo.net` live. Verified via green checks + live site below
- [x] 6.1 Updated `openspec/specs/authentication/architecture.md` (domain, preview hostnames, Secret Store env source, commit-SHA build var, callback URLs) and `AGENTS.md` env-setup notes
- [x] 6.2 `npm run test:all` green — lint ✓ typecheck ✓ spec:validate (11) ✓ unit+coverage (1522) ✓ e2e mock (121) ✓ storybook (31) ✓
- [ ] 6.3 **[needs live Cloudflare deploy]** Confirm prod-mode E2E passes against the live deployment and `/api/health` reports the correct commit on `codjiflo.net`
- [x] 6.3 Verified live: `https://codjiflo.net/api/health` reports the current `main` HEAD commit (`be52d898…`), and the required `e2e-tests-prod` check (which targets `codjiflo.net`) passes green on merged PRs
@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
codjiflo 11c0d47 Commit Preview URL

Branch Preview URL
Jun 02 2026, 03:35 AM

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 2, 2026

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 77.48% (🎯 70%)
⬆️ +0.18%
3210 / 4143
🔵 Statements 76.17% (🎯 70%)
⬆️ +0.20%
3366 / 4419
🔵 Functions 75.86% (🎯 70%)
⬆️ +0.21%
676 / 891
🟡 Branches 71.89% (🎯 70%)
⬆️ +0.15%
1901 / 2644
File CoverageNo changed files found.
Generated in workflow #1339 for commit 11c0d47 by the Vitest Coverage Report Action

@pedropaulovc pedropaulovc merged commit d7c45fb into main Jun 2, 2026
12 checks passed
@pedropaulovc pedropaulovc deleted the chore/cloudflare-migration-task-status branch June 2, 2026 03:36
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