Skip to content

Add GitHub CI workflows and dependency automation - #18

Merged
JessyTsui merged 5 commits into
mainfrom
feature/ci-codex-review
Mar 10, 2026
Merged

Add GitHub CI workflows and dependency automation#18
JessyTsui merged 5 commits into
mainfrom
feature/ci-codex-review

Conversation

@JessyTsui

@JessyTsui JessyTsui commented Mar 10, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • add baseline GitHub Actions workflows for frontend, backend, workers, migrations, preview deploys, and Codex review requests
  • cover the workers test suite in CI and rename the backend compile job so it does not claim type checking
  • harden PR automation by skipping Vercel preview deploys on forked pull requests and re-requesting Codex review per updated commit SHA
  • add docs/next-tasks.md to capture the remaining implementation tasks after the first sprint

Affected Directories

  • .github/workflows/
  • .github/dependabot.yml
  • docs/next-tasks.md

Config Changes

  • no new repository env vars were added
  • preview deploys expect existing VERCEL_TOKEN, VERCEL_ORG_ID, and VERCEL_PROJECT_ID repository secrets

Testing

  • pnpm --dir frontend lint
  • pnpm --dir frontend test
  • pnpm --dir frontend build
  • backend/.venv/bin/python -m pytest backend/tests -v
  • python3 -m pytest workers/tests -v
  • python3 -m compileall backend/app backend/tests workers
  • applied db/migrations/001_initial_schema.sql against a local pgvector/pgvector:pg16 Postgres container and verified the expected tables

Screenshots

  • not applicable

API Changes

  • not applicable

@vercel

vercel Bot commented Mar 10, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
cerul Ready Ready Preview, Comment Mar 10, 2026 10:14pm

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 99be128e2e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/preview-deploy.yml Outdated
preview:
name: Deploy Preview
runs-on: ubuntu-latest
if: github.event.pull_request.head.repo.fork == false

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge 在无 Secrets 的依赖 PR 中跳过预览部署

该 job 只用 github.event.pull_request.head.repo.fork == false 作为门槛,但仓库又在 .github/dependabot.yml 配置了 /frontend 的 Dependabot 更新;Dependabot 的 pull_request 工作流默认拿不到仓库 secrets,因此这里仍会执行到 vercel-token/vercel-org-id/vercel-project-id 并因空值失败,导致前端依赖升级 PR 持续红灯。

Useful? React with 👍 / 👎.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

已处理:在 preview deploy 中额外跳过 Dependabot PR,避免没有仓库 secrets 的依赖升级 PR 失败。修复在 4696376

Comment thread .github/workflows/migration-check.yml Outdated
PGDATABASE: cerul_test
run: |
echo "Running all migration files in order..."
for f in db/migrations/*.sql; do

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge 递归执行迁移目录下的 SQL 文件

触发条件监听的是 db/migrations/**,但实际执行只遍历 db/migrations/*.sql;如果后续把迁移按子目录组织(例如 db/migrations/2026/002.sql),工作流会被触发却不会应用这些文件,导致嵌套迁移中的 SQL 错误无法被这个检查捕获。

Useful? React with 👍 / 👎.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

已处理:迁移执行改成递归查找并按路径排序,能够覆盖 db/migrations 下的嵌套 SQL 文件。修复在 4696376

@JessyTsui
JessyTsui merged commit 754fbfc into main Mar 10, 2026
8 checks passed
@JessyTsui
JessyTsui deleted the feature/ci-codex-review branch March 24, 2026 20:16
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