Skip to content

fix: Astra 계약 정리 및 WorkGraph 저장 안정화 (v0.5.4) - #2

Merged
HyeonsangKim merged 1 commit into
mainfrom
codex/release-v0.5.4
Sep 7, 2026
Merged

HyeonsangKim merged 1 commit into
mainfrom
codex/release-v0.5.4

Conversation

@HyeonsangKim

Copy link
Copy Markdown
Contributor

Summary

GPT-6 Astra로 작업할 때 요청보다 큰 고정 절차를 적용하거나, 잘못된 증거 상태를 반환하던 경계를 정리합니다. 일반 코딩의 자동 스킬 호출 제한을 유지하고 필요한 계약만 적용합니다. 추가 축약 실험본은 채택하지 않았습니다.

  • WorkGraph의 read→revision 검사→save 전체를 잠가 동시 CLI 저장의 변경 유실을 막습니다. 오래된 expected revision은 충돌로 거부하고 잘못된 프로젝트 설정은 계획 저장 전에 거부합니다.
  • 읽기 전용 inspect, --root 기준 import 경로, 기호형 요구사항 ID, 한국어 화면 제목과 artifact 선택 처리를 개선합니다.
  • PRD·acceptance·delivery 지침에서 요청 범위와 증거 상태를 명확하게 하고 중복 절차·참조를 줄입니다. release-state는 untracked 내용을 읽지 않는 bounded diff를 제공합니다.
  • CLI usage의 미보고 항목은 unknown으로 유지하고, 오프라인 회귀 검사와 모델 평가 실행기·증거 검증기를 추가합니다. 필수 검증은 모델 서비스를 호출하지 않습니다.

상세 변경과 한계: v0.5.4 변경 내용과 검증 범위.

Verification

  • bash scripts/validate.sh: 원본 작업 폴더에서 통과
  • bash scripts/validate.sh: PR 파일만 담은 별도 worktree에서 통과
  • 기존 natural-language trigger/routing fixtures 통과; 일반 코딩 자동 호출 경계 유지
  • Plugin·marketplace 경로 및 두 manifest의 버전 일치 검증
  • git diff --check 및 변경된 README/EVALS/release 문서의 상대 링크 검사
  • 원시 세션 로그, 실험 작업 폴더, 기술보고서 초안과 무관한 사용자 변경 제외

추가 회귀에는 실제 CLI 두 프로세스의 변경 보존·revision 충돌·writer 종료 후 잠금 해제·dry-run 보존과 잘못된 프로젝트 설정의 preview/apply 거부가 포함됩니다.

Evidence boundaries

개발 중 Astra/medium 비교는 C8 지침을 포함한 42개 작업 단위·48개 CLI 호출이었습니다. 최종 C9 runtime은 별도 모델 비교 조건이 아니며, 사후 채점 정정과 미완료 실행을 포함한 개발 표본으로 일반 성능·비용 우위를 주장하지 않습니다. 이번 PR 준비에서 추가 모델 호출은 없습니다.

원시 연구 자료와 상세 보고서 초안은 로컬에 보존합니다. 이 PR은 공개 재현 자료 묶음이 아닙니다. 잠금은 협력하는 CLI 사이에만 적용되며 Windows 경로는 실측하지 않았습니다.

Release impact

  • Patch: Core와 Knowledge Wiki manifest 0.5.4
  • Knowledge Wiki 기능 변경 없음; 공동 버전 정책에 따른 manifest 변경만 포함
  • main 병합 시 기존 release workflow가 v0.5.4 태그와 GitHub Release를 생성할 수 있습니다. 이번 작업은 PR 생성까지이며 병합하지 않습니다.

@wigtn-contact

Copy link
Copy Markdown

판정: 승인

변경 파일

  • README.md
  • docs/EVALS.md
  • docs/RELEASE-v0.5.4-KO.md
  • plugins/wigtn-knowledge-wiki/.codex-plugin/plugin.json
  • plugins/wigtn-plugins-with-codex/.codex-plugin/plugin.json
  • plugins/wigtn-plugins-with-codex/references/evidence-contract.md
  • plugins/wigtn-plugins-with-codex/schemas/evidence-contract.schema.json
  • plugins/wigtn-plugins-with-codex/schemas/workgraph.schema.json
  • plugins/wigtn-plugins-with-codex/scripts/import-requirements.py
  • plugins/wigtn-plugins-with-codex/scripts/inspect-release-state.py
  • plugins/wigtn-plugins-with-codex/scripts/validate-evidence.py
  • plugins/wigtn-plugins-with-codex/scripts/validate-screen-spec.py
  • plugins/wigtn-plugins-with-codex/scripts/wigtn.py
  • plugins/wigtn-plugins-with-codex/scripts/workgraph_core.py
  • plugins/wigtn-plugins-with-codex/skills/acceptance-verifier/SKILL.md
  • plugins/wigtn-plugins-with-codex/skills/acceptance-verifier/references/evidence-matrix.md
  • plugins/wigtn-plugins-with-codex/skills/acceptance-verifier/references/saved-evidence.md
  • plugins/wigtn-plugins-with-codex/skills/design-direction/SKILL.md
  • plugins/wigtn-plugins-with-codex/skills/handdrawn-diagram/SKILL.md
  • plugins/wigtn-plugins-with-codex/skills/handdrawn-diagram/references/rendering.md
  • plugins/wigtn-plugins-with-codex/skills/product-spec/SKILL.md
  • plugins/wigtn-plugins-with-codex/skills/product-spec/references/create-contract.md
  • plugins/wigtn-plugins-with-codex/skills/product-spec/references/review-contract.md
  • plugins/wigtn-plugins-with-codex/skills/release-readiness/SKILL.md
  • plugins/wigtn-plugins-with-codex/skills/release-readiness/references/git-safety.md
  • plugins/wigtn-plugins-with-codex/skills/screen-spec/SKILL.md
  • plugins/wigtn-plugins-with-codex/skills/verified-delivery/SKILL.md
  • plugins/wigtn-plugins-with-codex/skills/verified-delivery/references/delivery-evidence.md
  • plugins/wigtn-plugins-with-codex/skills/work-planner/SKILL.md
  • plugins/wigtn-plugins-with-codex/skills/work-planner/references/workgraph-contract.md
  • scripts/check-codex-usage-missing.py
  • scripts/check-codex-usage.py
  • scripts/check-model-migration-eval.py
  • scripts/check-release-state.py
  • scripts/check-screen-contract.py
  • scripts/check-symbolic-requirement-ids.py
  • scripts/check-token-ablation.py
  • scripts/check-verified-delivery-routing.py
  • scripts/check-workgraph-concurrency.py
  • scripts/check-workgraph-import-root.py
  • scripts/check-workgraph-inspect.py
  • scripts/check-workgraph-project-guard.py
  • scripts/codex_usage.py
  • scripts/export-eval-packet.py
  • scripts/migration_cases.py
  • scripts/migration_evidence.py
  • scripts/run-behavior-evals.sh
  • scripts/run-model-migration-eval.py
  • scripts/run-ordinary-gate.sh
  • scripts/run-package-ablation.sh
  • scripts/run-static-contracts.sh
  • scripts/run-token-ablation.sh
  • scripts/run-workgraph-pilot.sh
  • scripts/score-behavior-smoke.py
  • scripts/score-model-migration-eval.py
  • scripts/score-token-ablation.py
  • scripts/summarize-token-efficiency.py
  • scripts/validate.sh
  • tests/behavior/token-efficiency-cases.tsv

확인한 것

  • Requirement ID import/schema/validator가 숫자형 및 REQ-A 형태를 일관되게 처리합니다.
  • WorkGraph inspect는 저장된 명령을 실행하지 않고, import 경로는 --root 기준으로 제한하며, apply 저장은 advisory lock으로 직렬화합니다.
  • 잘못된 project context에서 plan 저장을 거부하고 기존 graph를 보존합니다.
  • release-state diff는 staged/unstaged만 bounded하게 읽고 untracked 내용을 포함하지 않습니다.
  • 화면 계약의 한국어 제목과 공백/쉼표 artifact 선택을 검증합니다.
  • Codex usage 누락값은 0으로 추정하지 않고 unknown으로 보존합니다.
  • 차단급 동작 결함은 확인하지 못했습니다.

검증

  • 방법: CI
  • 명령/체크: gh pr checks 2 --repo wigtn/wigtn-plugins-codex
  • 결과: 통과
  • 근거: Repository contract check가 pass로 보고되었습니다. 추가로 bash scripts/validate.sh 및 관련 WorkGraph/usage/evidence 정적 검사를 로컬에서 실행해 모두 통과했지만, 최종 판정 근거는 CI 결과로 삼았습니다.

검토한 head SHA: 175c8e9a8bce19660e2f94ff68d2a99a8c662a36

이 리뷰가 도움이 됐으면 👍, 지적이 틀렸으면 👎 를 눌러주세요. 오탐 집계에 씁니다.

@wigtn-contact wigtn-contact 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.

변경 파일과 상세 검증 결과는 요약 코멘트를 확인해 주세요.

@HyeonsangKim
HyeonsangKim merged commit 89ca472 into main Sep 7, 2026
1 check 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