Skip to content

build: add locked local test entrypoint - #824

Merged
seonghobae merged 4 commits into
ContextualWisdomLab:mainfrom
seonghobae:fix/uv-locked-test-entrypoint
Aug 24, 2026
Merged

build: add locked local test entrypoint#824
seonghobae merged 4 commits into
ContextualWisdomLab:mainfrom
seonghobae:fix/uv-locked-test-entrypoint

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add make test as the local full-suite entrypoint
  • use the existing hash-locked CI pytest requirements through uv --no-project
  • document the reproducible command

Validation

  • make test (1435 passed)

Open in Devin Review

Summary by CodeRabbit

  • 테스트

    • make test 명령으로 전체 테스트를 간편하게 실행할 수 있습니다.
    • 테스트 실행 시 필요한 의존성이 자동으로 설치되며, 결과가 간결하게 표시됩니다.
  • 문서

    • 전체 테스트와 개별 스모크 점검의 실행 방법을 README에 안내했습니다.

@seonghobae
seonghobae enabled auto-merge (squash) August 23, 2026 09:34
@coderabbitai

coderabbitai Bot commented Aug 23, 2026

Copy link
Copy Markdown

Review Change Stack

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: cd10d1bf-ced0-4527-bdd9-dcaf6ef75695

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 58263099-b7c9-4a8a-9445-6e319ba4895c

📥 Commits

Reviewing files that changed from the base of the PR and between e226e11 and a421915.

📒 Files selected for processing (2)
  • Makefile
  • README.md

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

make test 대상을 추가했습니다. 이 대상은 uv로 퍼징 테스트 의존성을 설치한 뒤 pytest -q를 실행합니다. README.md에 전체 테스트와 개별 smoke check의 실행 방법을 설명했습니다.

Changes

테스트 실행 명령

Layer / File(s) Summary
테스트 명령 및 사용 안내
Makefile, README.md
Makefile.PHONY 테스트 대상을 추가했습니다. README.mdmake test와 기존 runtime lock을 사용하는 smoke check 실행 방법을 설명했습니다.

Estimated code review effort: 1 (단순) | ~5분

Merge Risk: ⚪ Minimal · up to a4219

This PR adds a reproducible local full-suite test command and documents its use; no actionable merge-blocking risk remains after normal checks and review.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 제목은 해시 잠금 기반의 로컬 테스트 진입점인 make test 추가를 명확하고 간결하게 설명합니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (2 skipped: 2 unsupported.)
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

devin-ai-integration[bot]

This comment was marked as resolved.

seonghobae and others added 3 commits August 24, 2026 00:13
…odels

The strix (quick) check on this exact head failed because NVIDIA NIM
rate-limited every configured fallback model (nemotron-3-super-120b,
llama-3.3-nemotron-super-49b, openai-direct/gpt-5.6-luna all hit 429/
connection errors) -- zero vulnerability report was produced, and Strix
correctly failed closed rather than treat that as a clean scan (this is
the same class of transient provider-availability failure already
diagnosed once this session on PR ContextualWisdomLab#768). Not a defect in this PR's actual
diff (Makefile + README only). Empty retry commit to trigger a fresh,
per-commit Strix run against the same tree.
Second retry: the first (af51113) hit the identical failure signature
1.5hrs after the original run -- NVIDIA NIM 429 across all configured
fallback models, zero vulnerability report, correct fail-closed. ~2hrs have
now passed since that retry; trying once more to check whether provider
capacity has recovered. Not a defect in this PR's diff (Makefile + README
only).
@seonghobae
seonghobae merged commit 50014ef into ContextualWisdomLab:main Aug 24, 2026
31 of 32 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.

1 participant