Skip to content

Preserve verified recovery and handoff steps - #18

Merged
ouyangyipeng merged 2 commits into
mainfrom
feat/recovery-handoff
Oct 23, 2026
Merged

ouyangyipeng merged 2 commits into
mainfrom
feat/recovery-handoff

Conversation

@ouyangyipeng

Copy link
Copy Markdown
Owner

Preserve a complete development handoff before the user deletes this machine's ClassFox checkout. Add a fresh-machine recovery guide, link it from README/docs navigation, and refresh release evidence and current project state.

A fresh clone from GitHub main 6c71657 was installed independently using the locked Python, desktop and mini-program dependencies. Passed: 202 backend tests and backend typecheck; 22 frontend tests/typecheck/build; 36 mini-program tests/typecheck/build; two bundle tests; 10 Edge browser scenarios; actual Python/Node loopback interop; strict docs build. These are fresh-checkout tests on the same Mac. Existing native build/frozen-backend CI 38866058139 remains applicable because no application, dependency or build code changed.

All local commits are already on remote branches, no stash exists, and v2.1.1's permanent remote release checksum manifest matches the three local installers. Public website/download links and both pinned domestic model sources remain accessible. Known issues #3/#6/#9/#12 and WeChat v2.5 acceptance work remain documented.

No personal database, configuration, credentials, model weights, dependency caches or build products are included. Local deletion awaits the user's second confirmation of exact targets and handling of personal classroom data, as their global rules require. Relevant docs/web checks and independent docs review precede rebase merge and Pages deployment.

Merge using Rebase and merge.

Root cause: NA
Solution: Document clean-clone setup, permanent release assets,
model sources, private data handling and remaining work before
removing the local checkout. Refresh the verified handoff status.
Risks: Personal databases and system credentials need separate
private migration; they are not part of the public source archive.
Dependency: Python 3.12, Node 24, pnpm 12.4.2, uv and Rust
Links: https://github.com/ouyangyipeng/ClassAssistant
Copilot AI balanced review requested due to automatic review settings October 23, 2026 10:26

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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 23, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-23T10:27:58.914881Z 7b29427 PR opened
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@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: 7b294272e9

ℹ️ 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 docs/getting-started/recovery.md Outdated

## 在新电脑恢复开发

准备 Python 3.12、uv、Node.js 24、pnpm 12.4.2 和 Rust;平台编译工具见[环境准备](environment.md)。随后执行:

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 Include OpenSSL in Intel Mac recovery prerequisites

On a fresh Intel Mac, the linked environment guide only asks for Xcode tools and Rust, but the immediately following uv sync must build cryptography 50 from source because no Intel wheel is available. As docs/getting-started/packaging.md:20 notes, this configuration requires Homebrew openssl@3; without adding that prerequisite here or to the linked environment guide, the documented recovery procedure fails before the development environment can start.

Useful? React with 👍 / 👎.

Root cause: Recovery setup omitted the OpenSSL library required by
  the locked cryptography source build on Intel Mac.
Solution: Add the Homebrew prerequisite before dependency sync in
  the recovery and environment guides, matching packaging guidance.
Risks: NA
Dependency: Homebrew openssl@3 on Intel Mac
Links: #18
@ouyangyipeng
ouyangyipeng merged commit f284f85 into main Oct 23, 2026
2 of 6 checks passed
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