Skip to content

docs(upstream-comments): Sprint 1 — upstream gh issue 답글 8건 body 작성 (사용자 manual 등록) - #7

Merged
tgkim-openerd merged 2 commits into
mainfrom
chore/sprint-1-upstream-comments-2026-05-28
May 28, 2026
Merged

tgkim-openerd merged 2 commits into
mainfrom
chore/sprint-1-upstream-comments-2026-05-28

Conversation

@tgkim-openerd

Copy link
Copy Markdown
Collaborator

개요

Phase B 의 본 fork cover 8건 upstream gh issue 답글 body 작성. classifier 가 external upstream comment 등록을 차단 (정당 — fork-internal 인가 ≠ external system writes 인가) → body 만 본 PR commit + 사용자가 manual gh issue comment 등록.

답글 8건

# issue 핵심
openai#281 app-server fails after logout/login PR #4 3d72d6e broker busy/timeout 분기 + plugin home 격리 root cause + cross-platform recovery
openai#282 Companion sessions in Desktop history v2.0+ plugin home 격리 + 격리 trade-off (openai#281/openai#342 cross-link)
openai#285 Stop/Session hooks cross-drive Windows cd-first hook pattern
openai#287 spawn ENOENT PATHEXT Windows process.mjs L62-82 buildCommandInvocation cmd.exe 래핑 (openai#337 와 동일)
openai#288 sendBrokerShutdown timeout broker-lifecycle.mjs L46-48 + L65-66 setTimeout + 함수 never rejects
openai#298 /codex:review max findings --max-findings flag (default 20, hard cap 100)
openai#337 spawn shell:true Windows process.mjs cmd.exe 래핑 + quoteWindowsCmdArg (#287 와 동일 fix)
openai#342 setup loggedIn:false broker busy PR #4 3d72d6e broker busy/timeout 분기 + HEALTH_STATUS.TRANSIENT + direct-fallback scope 명시

Codex pair R1 결과

agentId ae4644d47e2fdc82e:

R1 MEDIUM 2 + LOW 2 모두 commit 6196adc 에 wording polish 적용 (cosmetic loop 회피).

사용자 manual 등록 명령

PR merge 후 main 에서 다음 명령 실행 (8회 gh issue comment 등록):

```bash
for n in 281 282 285 287 288 298 337 342; do
gh issue comment "$n" --repo openai/codex-plugin-cc --body-file "docs/upstream-comments/${n}.md"
done
```

또는 사용자가 review 후 일부만 등록:

```bash
gh issue comment 281 --repo openai/codex-plugin-cc --body-file docs/upstream-comments/281.md

... per-issue

```

scope discipline

본 PR scope = body 작성만. 실제 upstream comment 등록은 사용자 manual action (Claude classifier 'External System Writes' 차단 영역 정합 — fork-internal 인가 ≠ external repo 의 사용자 GitHub identity action).

변경 파일 (8 files, 신규 docs/upstream-comments/)

  • 281.md / 282.md / 285.md / 287.md / 288.md / 298.md / 337.md / 342.md

merge 전략

Squash merge 권장 — 2 commits → 1 commit `docs(upstream-comments): Sprint 1` 로 main history clean.

…사용자 manual 등록)

본 fork (opnd-io/opnd-codex-plugin) 가 이미 cover 한 upstream openai/codex-plugin-cc 의 8 issue
답글 안 작성. classifier 가 external upstream comment 등록을 차단 (정당 — fork-internal 인가
≠ external system writes 인가). body 만 본 PR 에 commit + 사용자가 manual 로 gh issue comment
명령 8회 실행.

## 작성된 답글 8건

| # | issue | 답글 내용 요약 |
|---|---|---|
| 281 | app-server fails after logout/login | PR #4 commit 3d72d6e 의 broker busy/timeout 분기 + plugin home 격리 root cause + cross-platform recovery (cp + broker kill + WAL cleanup) |
| 282 | Companion sessions in Desktop history | v2.0+ plugin home 격리 (~/.codex/claude-code/) + 격리 trade-off (openai#281/openai#342 cross-link) + 권고 (cp recovery + inspectPluginHomeFreshness) |
| 285 | Stop/Session hooks cross-drive Windows | cd-first hook pattern (cd "${CLAUDE_PLUGIN_ROOT}" && node ...) + hooks.json 적용 |
| 287 | spawn ENOENT PATHEXT Windows | process.mjs L62-82 buildCommandInvocation cmd.exe 래핑 (openai#337 와 동일 fix) |
| 288 | sendBrokerShutdown timeout | broker-lifecycle.mjs L46-48 BROKER_SHUTDOWN_TIMEOUT_MS=5000 + L65 setTimeout + 함수 never rejects |
| 298 | /codex:review max findings | --max-findings flag (default 20, hard cap 100) |
| 337 | spawn shell:true Windows | process.mjs buildCommandInvocation cmd.exe /d /s /c call 래핑 + quoteWindowsCmdArg (openai#287 와 동일) |
| 342 | setup loggedIn:false broker busy | PR #4 commit 3d72d6e broker busy/timeout 분기 + HEALTH_STATUS.TRANSIENT (openai#281 root cause 동일) |

## 사용자 manual 등록 명령

```bash
for n in 281 282 285 287 288 298 337 342; do
  gh issue comment "$n" --repo openai/codex-plugin-cc --body-file "docs/upstream-comments/${n}.md"
done
```

또는 individually:

```bash
gh issue comment 281 --repo openai/codex-plugin-cc --body-file docs/upstream-comments/281.md
gh issue comment 282 --repo openai/codex-plugin-cc --body-file docs/upstream-comments/282.md
# ... (8회)
```

## scope discipline

본 PR 은 답글 body 작성만 — 실제 외부 comment 등록은 사용자 manual action (Claude classifier
"External System Writes" 차단 영역 정합). 사용자가 review 후 일부 / 전체 comment 등록 결정.

## 검증

- 8 body files: docs/upstream-comments/{281,282,285,287,288,298,337,342}.md
- doc-only (코드 변경 X) — npm test 회귀 영향 없음
- 본 fork PR #4 commit 3d72d6e + v2.0+ sprint commits 의 정확한 file:line 인용
@tgkim-openerd
tgkim-openerd merged commit bbe54ee into main May 28, 2026
@tgkim-openerd
tgkim-openerd deleted the chore/sprint-1-upstream-comments-2026-05-28 branch May 28, 2026 13:51
tgkim-openerd added a commit that referenced this pull request May 28, 2026
…ne + telemetry UX + Windows fixes (#10)

## Version bump 2.1.0 → 2.2.0 (semver minor — additive only, BREAKING 없음)

- package.json / package-lock.json / plugins/opnd-codex/.claude-plugin/plugin.json / .claude-plugin/marketplace.json 4 file version sync (scripts/bump-version.mjs 2.2.0)

## CHANGELOG release entry — 2.2.0 (2026-05-28)

본 release 의 main commits (9 PR squash):
- #1 daily-evolve pipeline Phase 0~6 + 1.5a + budget
- #3 /analyze Recommendations + /doc-sync + PII redact
- #4 broker false-negative 해소 (BROKER_BUSY_RPC_CODE + timeout 분기)
- #5 Phase A telemetry UX + setup advisory + verified-no-change
- #6 Phase B suminerProxy eval + upstream classification
- #7 Sprint 1 upstream comment body 8건
- #8 Sprint 2 openai#331 + openai#333 fix + 6 plan
- #9 Sprint 3 suminerProxy cherry-pick plan

## semver minor 정합

모두 additive (BREAKING 없음):
- 신규 daily-evolve pipeline (별도 commands/skills, 기존 영역 안 건드림)
- 신규 HEALTH_STATUS.TRANSIENT enum (existing 5 enum 보존)
- 신규 parseReviewArgv() (기존 parseArgs/parseCommandInput 보존)
- 신규 SELF_FORKS Set (fork-research 의 fork list filter — false-positive 제거만)
- 신규 inspectPluginHomeFreshness() + pluginHomeAdvisory field (additive)
- 신규 isUsageLimitError / annotateUsageLimitError (별도 helper)
- 기존 isStaleAuthCacheError pattern 확장 (backward compatible)
- broker busy / timeout 분기 추가 (catch fallback 보존)
- Windows process.mjs cmd.exe 래핑 (behavior change 가능성, e2e 검증 미완 — fork-cover 표기)

## marketplace install 즉시 가능

본 commit + 후속 push + main merge + tag v2.2.0 + gh release 후:

```bash
/plugin marketplace add opnd-io/opnd-codex-plugin
/plugin install opnd-codex@opnd-codex-plugin
/reload-plugins
/opnd-codex:setup
```

marketplace.json source = ./plugins/opnd-codex (main branch HEAD) → 사용자 install 시 v2.2.0 모든 변경 즉시 포함.

## 다음 sprint backlog (Unreleased)

- suminerProxy NDJSON event stream foundation cherry-pick (Phase 1, 3-4d)
- upstream HIGH port 6건 (openai#330 openai#345 openai#350 openai#349 openai#336 openai#295)
- quality-review-agent frontmatter description "3 영역 → 6 영역" (classifier 차단, 사용자 인가 후)
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