test: preserve optional-extra pytest collection

[seonghobae]

Product result

OpenCode coverage evidence can collect the runnable LineageWeave suite in a networkless environment that lacks optional backend extras. Hosted CI still installs the extras and collects the complete suite.

Exact stacked scope

• test: preserve optional-extra pytest collection #494 head: 5d9728a16051e7db453ca513cd5baa75be7450cc
• stacked fix: repair frontend build and publish ontology Pages #426 base: 2cf16db25d25ef90bf43737917a911ed8b840794 (worktree-fix-frontend-build-break)

The parent is now composed into the head by a normal merge. This fixes the prior synthetic-merge TypeScript failure while keeping the effective #494 diff exactly four optional-collection files: the root pytest hook, its standard-library AST helper, focused synthetic tests, and the changelog fragment. #426 owns the shared App/OIDC/Admin repair, ontology publication, and ADR 0001 gap baseline.

What this head changes

• Repository-root collection covers both tests and backend/tests.
• Exact parsed imports determine whether a test requires a missing optional module; comments and import-name prefixes cannot hide tests.
• A NumPy-only environment gap skips lineageweave.period_report consumers but not post_evaluation consumers.
• Kept paths return None, preserving pytest built-in and project ignore rules.
• Lazy Redis imports and unreadable or non-UTF-8 paths remain collectable so real collection errors stay visible.

Validation

• Exact-head focused helper suite: 10 passed.
• Exact-head frontend lint and production build passed after parent composition.
• Prior unchanged-helper evidence: changed production statement/branch coverage 100%, relevant callable docstrings 15/15, isolated dev-only suite 406 passed / 7 skipped, and the explicit --ignore collection omitted the requested helper suite.
• Effective PR diff: four files; diff hygiene passed.

Protected stack lifecycle

#426 must reach protected main first. Then retarget #494 to main and obtain fresh exact-head checks, threads, and independent approvals; no evidence from this unprotected stack base transfers. Auto-merge remains disabled while stacked. No self-approval, admin bypass, force push, provider credential, real record, or PII is included.

[seonghobae]

Verification comment only — not an approval (this identity cannot self-approve).

Exact head 077ec72:

• Unauthenticated login no longer mounts Admin settings with an undefined token.
• Login uses returnUrlFromLocation + rememberOidcReturnUrl.
• Local tsc -b passed; App + OIDC tests: 83 passed.
• Gap baseline is non-identifying and bound to the live PR/issue inventory.

Independent OpenCode / Strix / Noema exact-head review is still required before squash-merge.

[seonghobae]

Independent exact-head review requested (OpenCode / Strix / Noema). This identity cannot self-approve.

Local verification on 077ec72:

• pnpm exec tsc -b passed
• pnpm exec vitest run src/App.test.tsx src/oidcReturnUrl.test.ts — 83 passed

Please review this exact head. Squash-merge only after independent approval and current checks.

[coderabbitai]

📝 Walkthrough

Walkthrough

선택적 백엔드 모듈이 설치되지 않은 환경에서 관련 테스트 수집을 건너뛰도록 했습니다. AST 기반 import 감지 헬퍼와 pytest 수집 훅을 추가하고, 직접·전이 import 및 예외 경로를 테스트했습니다.

Changes

선택적 추가 기능 수집 제어

Layer / File(s)	Summary
선택적 모듈 및 import 감지 lineageweave/optional_extra_collection.py, tests/test_optional_extra_collection.py	선택적 모듈 누락 여부를 확인합니다. AST로 직접 및 알려진 로컬 모듈을 통한 전이 import를 감지합니다. 주석, 비Python 경로, 헬퍼 자체 테스트는 제외하지 않습니다.
pytest 수집 훅 연결 conftest.py, CHANGELOG.d/2.12.7-optional-extra-collection.md	pytest_ignore_collect 훅이 누락된 선택적 모듈을 요구하는 테스트 경로의 수집을 건너뜁니다. 변경 범위와 CI 동작을 changelog에 기록합니다.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Merge Risk: 🟡 Moderate · up to 385c6

The added optional collection path can fail test collection with a missing numpy dependency mapping in a supported environment. This bounded integration issue should be fixed or explicitly accepted before merge.

Sequence Diagram(s)

sequenceDiagram
  participant Pytest
  participant conftest.py
  participant optional_extra_collection
  participant TestPath
  Pytest->>conftest.py: pytest_ignore_collect 호출
  conftest.py->>optional_extra_collection: 누락 모듈 및 경로 의존성 조회
  optional_extra_collection->>TestPath: Python 파일 import 분석
  optional_extra_collection-->>conftest.py: 수집 제외 여부 반환
  conftest.py-->>Pytest: 경로 수집 또는 건너뛰기

Loading

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	Docstring check was indeterminate for this PR — some files could not be analyzed in time. Not blocking.
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.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	제목은 선택적 추가 기능이 없을 때 pytest 테스트 수집을 유지하는 실제 변경 사항을 정확히 요약합니다.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/frontend-adminpanel-unauthenticated-build

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[devin-ai-integration]

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

[seonghobae]

@opencode-agent independent exact-head review requested. This identity cannot self-approve.

Head 077ec7289360459e9585a6c0d67b0d64feceb14d:

• Unauthenticated login no longer mounts Admin settings with an undefined token (tsc -b green).
• Login persists a validated OIDC return URL.
• Gap baseline is non-identifying and bound to the live PR/issue inventory.
• Local App + OIDC tests: 83 passed. Hosted Frontend lint/test/build and Full test suite succeeded on this head.

Please review this exact head. Squash-merge only after independent approval (OpenCode + one other independent reviewer) and current checks.

[seonghobae]

@opencode-agent independent exact-head review requested after the gap-baseline bind to ContextualWisdomLab/.github#1258 and #1259. This identity cannot self-approve.

Still the login tsc repair plus non-identifying baseline. Auto-merge remains armed.

[seonghobae]

Cycle retry (2026-08-23T14:07Z KST+9): write probe from seonghobae. Independent exact-head APPROVE still required; author will not self-approve. Do not squash-merge without independent APPROVE plus exact-head required success. Issues #79 and #87 stay open.

[seonghobae]

Closing as superseded after an exact-head comparison, rather than creating a third competing baseline writer.

• fix: repair frontend build and publish ontology Pages #426 at c8a4be8fc2417f05d53fb68d32d9e59c3d443e25 carries the same App.tsx unauthenticated-admin fix plus the OIDC helper and App integration coverage, and remains the ontology publication carrier.
• docs: restore non-identifying gap baseline #479 at c0ac6f5dbc691688fe2b7ba8a79e18d6d5264dd8 carries the newer non-identifying product/technical gap baseline.
• The three unresolved baseline findings on this PR are valid: the 52-head inventory was incomplete, the package ordering was improperly normative without an ADR, and issue [Project history] Add authoritative lifecycle ingestion and idempotent reconciliation #284 was incorrectly self-mapped as a PR. Those defects are not being merged or copied into the canonical baseline.

No review was dismissed or self-approved, and no protected merge was attempted.

[seonghobae]

Reopened this cycle: login tsc repair is still missing on protected main (ef6f5a5), so every frontend PR inherits the unauthenticated AdminPanel poison. Independent exact-head APPROVE still required; author will not self-approve. Do not squash-merge without independent APPROVE plus exact-head required success. Issues #79 and #87 stay open.

#426 remains the ontology Pages carrier. This head stays the smallest shared-gate login/tsc repair plus non-identifying gap baseline.

[seonghobae]

@opencode-agent current-head review for 7eb5b2a (login typecheck only). Gap baseline is LineageWeave#497. This identity cannot self-approve.

[seonghobae]

@opencode-agent @cwl-noema-review current-head review for 7eb5b2a89a6f32785bbbaf89126cb1ba931a03a8.

Login-only unique value: returnUrlFromLocation + rememberOidcReturnUrl; unauthenticated shell does not mount Admin settings. Gap baseline is owned by LineageWeave#426 after #497 merged into that branch; this head must not fold the docs back in. Auto-merge is armed. This identity cannot self-approve.

[seonghobae]

@opencode-agent @cwl-noema-review current-head review for 7eb5b2a89a6f32785bbbaf89126cb1ba931a03a8. Login tsc only. Independent OpenCode / Strix / Noema required. This identity cannot self-approve.

[seonghobae]

@opencode-agent @cwl-noema-review current-head review for 327c359b35add7b8011542fb5bd07e625de41e29.

Restored the login tsc repair on this head so Frontend lint/test/build is not poisoned by protected main. Optional-extra collection remains. Gap baseline stays on #426. Independent OpenCode / Strix / Noema required. This identity cannot self-approve.

[seonghobae]

@opencode-agent @cwl-noema-review current-head review for b195b42dd327f6a6659cca315b34886ab11d38ed.

Restored login tsc (unauthenticated AdminPanel removed; OIDC return-URL helpers used). Optional-extra collection remains. Gap baseline stays on #426. Independent OpenCode / Strix / Noema required. This identity cannot self-approve.

[seonghobae]

@opencode-agent @cwl-noema-review current-head review for b195b42dd327f6a6659cca315b34886ab11d38ed. Login tsc passing on hosted Frontend lint/test/build. Optional-extra collection remains. Gap baseline stays on #426. Independent OpenCode / Strix / Noema required. This identity cannot self-approve.