Skip to content

release: merge dev into main#197

Open
quan0715 wants to merge 30 commits into
mainfrom
dev
Open

release: merge dev into main#197
quan0715 wants to merge 30 commits into
mainfrom
dev

Conversation

@quan0715

Copy link
Copy Markdown
Owner

Summary

  • Promote the latest dev branch to main.
  • Includes auth provider consolidation, QAuth/NYCU login documentation and showcase updates, and score precision fixes.
  • Includes the exam monitoring/auth refresh changes already merged into dev.

Verification

  • PR feat: refresh auth providers and score precision #196 CI green before merging into dev: Static Analysis & Build, Backend Unit Tests, Frontend Unit Tests, Integration Tests, Judge Tests, MCP Server Tests.
  • Local frontend coverage passed: npm run test:coverage.
  • Local pre-push checks passed for branch codex/auth-experience-refresh; E2E remains skipped by hook as configured.

quan0715 and others added 29 commits May 20, 2026 22:06
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…dent CSV

- Add exam3-subjective preset (18 short-answer questions, contest d4edfae6)
- Register deepseek-v4-flash and deepseek-v4-pro as canonical model IDs in
  model_factory, serializer choices, and credits pricing table
- Include exam3 grading results: summary.csv + per_student_scores.csv
  (118 students, 51pt max, avg 36.30, with recovery runs for Q2/Q7/Q10/Q11/Q12)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
# Conflicts:
#	docs/superpowers/specs/2026-05-20-ai-grading-experiment-cli-design.md
#	docs/superpowers/specs/2026-05-20-qjudge-paper-cli-oauth-first-design.md
#	frontend/src/features/contest/screens/paperExam/hooks/usePaperExamAutoSave.test.ts
#	frontend/src/features/contest/screens/paperExam/hooks/usePaperExamAutoSave.ts
Align auth entry pages with the refreshed visual shell, remove legacy login hero code, and switch the auth illustration asset to WebP.
Refresh auth entry responsive layout, generic campus registration flow, and auth i18n cleanup.
Filter campus SSO choices by registration support on the registration flow while preserving login SSO provider visibility.
feat: refresh auth providers and exam monitoring

Squashed from PR #195 after all CI checks passed.
Merge auth provider consolidation, QAuth documentation/showcase updates, and fixed score precision handling.
Copilot AI review requested due to automatic review settings June 26, 2026 12:22

Copilot AI 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.

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@chatgpt-codex-connector

Copy link
Copy Markdown

💡 Codex Review

A141087林哲勗,38.0,51,1.0,2.0,2.0,3.0,4.0,2.0,1.0,2.0,2.5,3.0,3.0,1.0,3.0,0.5,2.0,2.0,4.0,0.0

P1 Badge Remove committed student-identifying grade exports

This generated export contains student identifiers/names alongside total and per-question scores. When this commit is pushed to a shared or public repo, those grades become available to anyone with repo access and are hard to purge from history. Please remove these reports from the commit or replace them with anonymized fixtures.


kwargs: dict[str, Any] = {"timeout": self.docker_timeout}
try:
client = docker.DockerClient(**kwargs)

P2 Badge Restore DOCKER_HOST support for the judge

In deployments that set DOCKER_HOST (still documented in example.env) to point the judge at a non-default Docker daemon/socket, kwargs now only contains timeout and the setting is no longer defined, so docker.DockerClient(**kwargs) falls back to the default local socket and judging fails when that socket is unavailable. Please keep passing base_url from env/settings or update the deployment config/docs.


# AUTH_PROVIDER_OPTIONS_JSON=[{"key":"nycu","type":"oidc","category":"campus","display_name":"NYCU 國立陽明交通大學","display_name_i18n_key":"auth.providers.nycu","logo_url":"/illustrations/nycu-logo.png"}]

P2 Badge Wire the documented auth provider override

The new example advertises AUTH_PROVIDER_OPTIONS_JSON as an override for public login method metadata, but /api/v1/auth/options ultimately returns the hard-coded provider registry and no backend code reads this env var. In deployments that set it to hide unconfigured providers or customize labels/logos, the frontend will ignore the override and still render the built-in provider list. Please implement the setting or remove it from the examples.

ℹ️ 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".

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