Skip to content

Conversation

@code-yeongyu
Copy link
Owner

Summary

README 파일 구조를 개선하여 가독성과 사용자 경험을 향상시킵니다.

Changes

Structure Improvements

  • TL;DR 위치 변경: "Oh My OpenCode" 소개 직후, Installation 직전으로 이동하여 빠른 개요 제공
  • For LLM Agents 섹션 토글화: <details> 태그로 감싸 가독성 개선
  • 설정 섹션 간소화: 장황한 스키마 설명 제거, 핵심만 유지

Content Improvements

  • Model Override 예시 위치 수정: Configuration 섹션에서 Features > Agents 섹션 하단으로 이동
  • context7 MCP 추가 (한국어 README): 누락된 MCP 설명 추가

Verification Checklist

  • TL;DR is now before Installation
  • LLM Agent section is collapsible
  • Schema explanation simplified
  • Model override example in correct location
  • Both EN/KO READMEs updated consistently

- Move TL;DR section before Installation for quick overview
- Wrap 'For LLM Agents' section in collapsible details tag
- Simplify Configuration section (remove verbose schema explanations)
- Move model override example to Features > Agents section
- Add context7 MCP to Korean README
@code-yeongyu code-yeongyu merged commit 4b7a4b0 into master Dec 5, 2025
1 check passed
@code-yeongyu code-yeongyu deleted the readme-structure-improvement branch December 21, 2025 09:13
sssgun pushed a commit to sssgun/oh-my-opencode that referenced this pull request Jan 18, 2026
- Move TL;DR section before Installation for quick overview
- Wrap 'For LLM Agents' section in collapsible details tag
- Simplify Configuration section (remove verbose schema explanations)
- Move model override example to Features > Agents section
- Add context7 MCP to Korean README
qwertystars added a commit to qwertystars/oh-my-opencode that referenced this pull request Jan 29, 2026
- Remove sisyphusJuniorModel fallback bypass (BUG #1)
  - userModel fallback was incorrectly bypassing category fallbackChain
  - Now correctly uses category requirements before system default

- Fix variant preservation in actualModel fallback path (BUG #2)
  - Variant was lost when categoryModel undefined but actualModel existed
  - Now preserves variant when falling back to actualModel parsing

- Add variant to session continuation (BUG code-yeongyu#3)
  - Session resume didn't pass variant from message history
  - Now extracts resumeVariant and passes as body.variant

- Remove unused inheritedModel parameter (BUG code-yeongyu#4)
  - resolveCategoryConfig had misleading unused parameter
  - Cleaned up function signature and all call sites

- Add variant-model compatibility validation (BUG code-yeongyu#5)
  - No warning when variant might not be supported by provider
  - Added isVariantLikelySupported() validation with console.warn

Test coverage:
- Updated 10+ existing tests to remove inheritedModel references
- Added 9 new tests for variant validation functions
- All 110 tests pass (77 delegate-task + 33 model-requirements)
sadnow referenced this pull request in sadnow/oh-my-opencode Feb 3, 2026
Task 4 complete - Wave 2:
- Apply normalizeModelID() in parseModelRef (tiers.ts)
- Apply normalizeModelID() in extractModelName (token-estimator.ts)
- Update test expectations for normalized outputs
- Fixes Bug #4: Model name inconsistencies

All normalization tests passing. Backward compatible.
marlon-costa-dc added a commit to marlon-costa-dc/oh-my-opencode that referenced this pull request Feb 11, 2026
…ate on plan approval

- Change mcb.* to /mcb.* in .gitignore to avoid ignoring source files like src/config/schema/mcb.ts
- Add resumeTeammateWithMessage() call after plan_approved and plan_rejected paths in send-message-tool.ts
- Fixes PR review comments code-yeongyu#3 and code-yeongyu#4 from cubic-dev-ai
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