Repository navigation
docs: retire Docker, Nginx, and finished plans; document Vercel hosting - #74
Merged
Merged
Conversation
Production runs on Vercel, so the Dockerfile, nginx.conf, and .dockerignore go, along with the Nginx assertions in the Vercel config test. The quality workflow is renamed from "Build and Push Docker Image" to "Quality", and the staging workflow's trigger follows the new name. Removes plans that have shipped, old evidence media and traces, SEO work logs, pricing snapshots, the old security scan, and .audit checklists. The Current Debate plans and the on-hold Workspace plans stay in .hermes/plans. Rewrites README as a short overview and updates AGENTS.md, the docs index, architecture, operations, testing, development, and the Supabase README for Vercel hosting, Current Debate, hidden Workspace, and local verification while GitHub Actions is unavailable. plans/README.md now lists in-progress, backlog, on-hold, and shipped work. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01L6qeHSDkqzoR9piKnn9XW3
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
There was a problem hiding this comment.
Programcode has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Free Run ID: ⛔ Files ignored due to path filters (22)
📒 Files selected for processing (57)
💤 Files with no reviewable changes (33)
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe pull request removes Docker and Nginx packaging, updates workflow and hosting documentation to describe Vercel, revises repository and plan guidance, and deletes selected implementation plans and historical records. ChangesHosting and repository documentation
Estimated code review effort: 3 (Moderate) | ~25 minutes Security Architecture ReviewSecurity architecture risk: 🔵 Low · up to No new security exposure was identified. The hosting configuration and protected staging conditions appear preserved, but deployed behavior during failure or rollback was not directly verified. Retained concerns Security review detailsSecurity Blast Radius
Trust Boundaries and Controls
Resilience and Maintainability Implications
Hardening Proposals
Note 🎁 Summarized by CodeRabbit FreeYour organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Essentials by visiting https://app.coderabbit.ai/settings/billing. Comment |
Moves the four kept plans from .hermes/plans/ into plans/, deletes .hermes, and points every reference at the new location. The docs checker no longer needs a separate .hermes exclusion. AGENTS.md and the operations guide now say that Actions jobs failing within 2 to 5 seconds with no logs never ran because of billing, and that the same checks must then run locally with the results recorded in the PR. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01L6qeHSDkqzoR9piKnn9XW3
There was a problem hiding this comment.
Programcode has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.
Owner
Author
|
@coderabbitai review |
✅ Action performedReview finished.
|
Owner
Author
|
Local verification at 6ccec31. GitHub Actions didn't run: every job failed in 2–3 seconds with no logs, which is the billing block.
Generated by Claude Code |
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Outcome
The docs now match how DebateLab runs today: Vercel hosting, Current Debate, a hidden Workspace, and a clear rule for when GitHub Actions is blocked by billing. All plans now live in
plans/, and.hermes/is gone.AGENTS.mdandREADME.mdwere the priority.Removed
Dockerfile,nginx.conf,.dockerignore, and the Nginx assertions insrc/test/vercel-deployment-config.test.ts. Everyvercel.jsonassertion stays.docs/performance/,docs/evidence/,docs/superpowers/,docs/pricing-snapshots/,docs/decisions/, the three 2026-07-05 SEO logs,security/scans/, and.audit/. That's about 12 MB, mostly videos and JSON traces. Git history keeps all of it.Moved
.hermes/plans/toplans/, and.hermes/is deleted. References inside those plans and across the docs point to the new paths.CI change
deploy.ymlis renamed from "Build and Push Docker Image" toQuality, since #73 already removed the Docker job.staging-e2e.ymlstarts off that workflow's name, so itsworkflow_runtrigger and error text are updated too. The job names (quality,database-tests,browser-tests) are unchanged, so the required checks onmainstill match.Rewritten
README.mdis now a short overview: features, quick start, checks, how the pieces fit, the repository map, and doc links.AGENTS.mdnow covers:maindeploys the frontendplans/as the only plan locationdocs/operations.mdreplaces the Docker/Nginx section with a Vercel hosting section. It also gains a section on what to do when Actions can't run, and Vercel rollback steps.docs/architecture.md,docs/testing.md,docs/development.md,supabase/README.mdanddocs/README.mdget targeted fixes.plans/README.mdnow lists what's in progress (PR 2), the backlog, what's on hold, and what shipped, with PR numbers.scripts/check-docs.mjsno longer excludes the deleted folders, and its test is updated to match.Verification (local)
At dfb8a62:
npm run docs:check,npm run lintandnpm run typecheckpassed.npm test -- --coverage: 108 files and 1,074 tests passed.npm run buildpassed.At 6ccec31:
npm run docs:checkandnpm run lintpassed..hermesreferences found only the home-directory token path inscripts/gsc_submit.py, which is unrelated.Not run: Playwright E2E, the database gate, and GitHub Actions (blocked by billing). No runtime code changed.
🤖 Generated with Claude Code
https://claude.ai/code/session_01L6qeHSDkqzoR9piKnn9XW3
Summary by CodeRabbit