Skip to content

feat: document deployment plan and harden ci - #9

Merged
KazanderDad merged 3 commits into
mainfrom
codex/complete-session-9-in-agent-instructions
Nov 9, 2025
Merged

feat: document deployment plan and harden ci#9
KazanderDad merged 3 commits into
mainfrom
codex/complete-session-9-in-agent-instructions

Conversation

@KazanderDad

Copy link
Copy Markdown
Member

Summary

  • add a deployment & operations playbook and record session 09 updates across the specs, addresses ledger, and logs
  • replace the placeholder GitHub Action with a full pnpm/Foundry lint-build-test workflow and commit the pnpm lockfile plus workspace build hints
  • update the frontend build pipeline (webpack flag, PostCSS config, lazy Supabase client, env example) and refresh the associated tests/mocks

Testing

  • pnpm lint
  • CI=1 pnpm --filter frontend build
  • pnpm --filter indexer build
  • pnpm build
  • pnpm --filter frontend test
  • pnpm --filter indexer test
  • pnpm test

Codex Task

Copilot AI review requested due to automatic review settings November 9, 2025 14:04

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR establishes deployment and CI/CD readiness for the peer-mapper project by adding production-grade infrastructure configuration, implementing a singleton pattern for the Supabase client, and creating comprehensive deployment documentation.

  • Converted Supabase client from static export to singleton factory pattern
  • Implemented full CI/CD pipeline with linting, building, and testing across all workspaces
  • Added deployment playbook and environment configuration examples

Reviewed Changes

Copilot reviewed 16 out of 18 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
.github/workflows/ci.yml Replaced placeholder CI with comprehensive quality gate including lint, build, and test steps for all workspaces
frontend/src/lib/supabaseClient.ts Refactored from static client export to singleton factory pattern with lazy initialization
frontend/src/lib/profile.ts Updated to use getSupabaseClient() factory function
frontend/src/lib/auth.ts Updated to use getSupabaseClient() factory function
frontend/__tests__/profile.test.ts Updated mock to match new factory function signature
frontend/postcss.config.mjs Changed from import-based to object-based plugin configuration
frontend/package.json Added --webpack flag to build command
frontend/.env.example Added example environment variables for frontend configuration
frontend/.gitignore Added exception for .env.example files
.gitignore Removed pnpm-lock.yaml from ignore list and added .env.example exception
pnpm-workspace.yaml Added onlyBuiltDependencies configuration for native modules
agent-context/deployment.md New comprehensive deployment playbook with detailed instructions
agent-context/technical-spec.md Updated deployment section with CI/CD guidance
agent-context/functional-spec.md Added deployment and operations section
agent-context/eas-addresses.md Enhanced with Moonscan verification URLs and deployment transaction hashes
agent-context/session-logs/session-09.md Added session log documenting deployment readiness work
agent-context/session-log.md Updated with session 09 summary

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread pnpm-workspace.yaml
- frontend
- indexer
- contracts
onlyBuiltDependencies:

Copilot AI Nov 9, 2025

Copy link

Choose a reason for hiding this comment

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

Corrected spacing: missing space after colon. Should be 'onlyBuiltDependencies: ' with proper YAML spacing.

Copilot uses AI. Check for mistakes.
Comment thread pnpm-workspace.yaml Outdated
- better-sqlite3
- esbuild
- sharp
- unrs-resolver

Copilot AI Nov 9, 2025

Copy link

Choose a reason for hiding this comment

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

Potential typo in package name 'unrs-resolver'. This package does not appear to exist in npm registry. Did you mean 'unrs' or another package? Verify this is the correct package name.

Suggested change
- unrs-resolver

Copilot uses AI. Check for mistakes.
Comment thread .github/workflows/ci.yml Outdated
run: pnpm --filter indexer build

- name: Build contracts
run: pnpm build

Copilot AI Nov 9, 2025

Copy link

Choose a reason for hiding this comment

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

This step runs 'pnpm build' at the workspace root without a filter, which may have unclear scope. Consider using 'pnpm --filter contracts build' to match the pattern of the frontend and indexer build steps for consistency and clarity.

Suggested change
run: pnpm build
run: pnpm --filter contracts build

Copilot uses AI. Check for mistakes.
Comment thread .github/workflows/ci.yml Outdated
run: pnpm --filter indexer test

- name: Test contracts
run: pnpm test

Copilot AI Nov 9, 2025

Copy link

Choose a reason for hiding this comment

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

This step runs 'pnpm test' at the workspace root without a filter, which may run tests across all packages or none. Consider using 'pnpm --filter contracts test' to match the pattern of the frontend and indexer test steps for consistency and clarity.

Suggested change
run: pnpm test
run: pnpm --filter contracts test

Copilot uses AI. Check for mistakes.
@KazanderDad
KazanderDad merged commit 4c9213c into main Nov 9, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants