Skip to content

[WRONG BRANCH] chore(fork): Windows-first 維護骨架 - #1

Merged
SanHsien merged 4 commits into
mainfrom
fork/windows-maintenance-overlay
Aug 22, 2026
Merged

[WRONG BRANCH] chore(fork): Windows-first 維護骨架#1
SanHsien merged 4 commits into
mainfrom
fork/windows-maintenance-overlay

Conversation

@SanHsien

Copy link
Copy Markdown
Owner

Summary

  • Fork overlay for SanHsien/opencodex, tracking the ChatGPT account pool product from lidge-jun/opencodex.
  • Isolate official npm publish, GitHub Pages, and issue/PR governance workflows so they only run on lidge-jun/opencodex.
  • Add Windows gate (tools/dev_check.ps1), overlay docs, weekly upstream review, and CodeQL.

Test plan

  • bun install --frozen-lockfile
  • pwsh -NoProfile -File tools\dev_check.ps1 (14 fork tests + link check)
  • bun run typecheck
  • bun tools/check-upstream-updates.ts (no new commits)
  • GitHub Actions: fork-maintenance, CodeQL, and upstream ci.yml on this PR
  • After merge: bun run src/cli/index.ts start and open http://localhost:10100 to configure providers / account pool

Keep the upstream ChatGPT account-pool product intact, isolate official npm/Pages and governance workflows to lidge-jun/opencodex, and add a local Windows gate for overlay docs and upstream review.

Co-authored-by: Cursor <cursoragent@cursor.com>
Copilot AI lite review requested due to automatic review settings August 22, 2026 10:42

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 was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions

Copy link
Copy Markdown

Deterministic PR hygiene checks passed.

@github-actions github-actions Bot added the chore label Aug 22, 2026
@github-actions github-actions Bot changed the title chore(fork): Windows-first 維護骨架 [WRONG BRANCH] chore(fork): Windows-first 維護骨架 Aug 22, 2026
@github-actions

github-actions Bot commented Aug 22, 2026

Copy link
Copy Markdown

⏳ DRAFT

  • wrong target branch (main); retarget to dev. UI screenshot required.

What to do

  • Retarget this PR to dev — all contributions go to dev.
  • Add a screenshot of the UI change to the PR description.

Its title has been prefixed with [WRONG BRANCH].
This pull request was already a draft. Its draft status will be preserved after every issue above is resolved.

@github-actions
github-actions Bot marked this pull request as draft August 22, 2026 10:42

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

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

throw new UpstreamCheckError(`unexpected git log line: ${JSON.stringify(line)}`);
}
const [sha, date, subject] = parts;
const files = runGit(["show", "--name-only", "--format=", sha], repoDir)

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 Diff merge commits against their first parent

When upstream main receives a merge commit, git show --name-only --format= uses combined-merge semantics and can omit nearly every changed file. For example, the existing release merge ceed6bf5 reports only package.json here, while its first-parent diff contains 224 paths, so the generated review report materially understates the affected surface without even showing a … +N more warning. Use a first-parent diff such as git show --first-parent --name-only --format= <sha> (or an equivalent diff-tree invocation) for merge commits.

Useful? React with 👍 / 👎.

SanHsien and others added 2 commits August 22, 2026 18:59
Bare gh pr create on a fork clone targets the parent repo. Record the SanHsien-only default and the single exception: explicit contribute-back consent in the current conversation.

Co-authored-by: Cursor <cursoragent@cursor.com>
Cursor 的規則是 per-project 的,先前只有 opencodex 有這條,其餘 fork 都沒有。
內容與 AGENTS.md、Codex 的 ~/.codex/AGENTS.md、Antigravity 的 ~/.gemini/GEMINI.md
一致:gh 在 fork clone 的預設 repo 就是上游,所以每個 clone 先 gh repo set-default,
開 PR 明寫 --repo/--base/--head,建完讀輸出 URL 確認 owner;回貢上游要主人當次
對話明確同意。2026-08-22 一天內兩個工具各誤開一個上游 PR。

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@github-actions github-actions Bot added documentation Improvements or additions to documentation and removed chore labels Aug 22, 2026
README.md is the Traditional Chinese product page; README.en.md is English. GUI and docs-site retain only en and zh-TW.

Co-authored-by: Cursor <cursoragent@cursor.com>
@SanHsien
SanHsien merged commit e083d46 into main Aug 22, 2026
@SanHsien
SanHsien deleted the fork/windows-maintenance-overlay branch August 22, 2026 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants