Skip to content

feat(env): env 계약 설정 추가#110

Merged
JeremyDev87 merged 1 commit into
masterfrom
feature/104-env-contract-config
May 6, 2026
Merged

feat(env): env 계약 설정 추가#110
JeremyDev87 merged 1 commit into
masterfrom
feature/104-env-contract-config

Conversation

@JeremyDev87

@JeremyDev87 JeremyDev87 commented May 5, 2026

Copy link
Copy Markdown
Owner

배경

#104는 CI/hosting 주입 env와 선택적 로컬 env를 repo 설정으로 선언해 env-missing-concrete 경고 노이즈를 줄이는 작업입니다.

변경 사항

  • maximus.config.json에 env.ciInjectedKeys, env.optionalLocalKeys schema를 추가했습니다.
  • env-missing-concrete 계산에서 설정된 exact key를 제외하도록 env check에 config를 전달했습니다.
  • unknown env config field가 실패하도록 deny_unknown_fields 회귀를 추가했습니다.
  • config 미사용 기본 동작과 CLI audit 적용 경로를 테스트로 고정했습니다.

검증

  • cargo test -p maximus-core --test config_loader
  • cargo test -p maximus-checks --test env_checks
  • cargo test -p maximus-cli --test config_and_filtering
  • cargo test -p maximus-core
  • cargo test -p maximus-checks
  • cargo test -p maximus-cli
  • cargo test --workspace --no-run
  • npm test
  • git diff --check
  • fresh-session Devils Advocate review: Critical 0 / High 0 / Medium 0 / Low 0

브랜치 / 워크트리

  • base: master
  • head: feature/104-env-contract-config
  • worktree: /private/tmp/maximus-issue104-env-contract-config

이슈 연결

Closes #104

env.ciInjectedKeys와 env.optionalLocalKeys로 env-missing-concrete 대상에서 CI 주입 및 선택 로컬 키를 제외한다.

기본 동작과 unknown field 실패 계약은 회귀 테스트로 고정한다.

Closes #104
@JeremyDev87 JeremyDev87 added enhancement New feature or request test labels May 5, 2026
@JeremyDev87

Copy link
Copy Markdown
Owner Author

독립 fresh-session 리뷰 결과

  • Verdict: APPROVE
  • Severity Summary: Critical 0 / High 0 / Medium 0 / Low 0
  • Findings: 없음
  • Rationale: env.ciInjectedKeys / env.optionalLocalKeys 스키마는 deny_unknown_fields 계약을 유지하고, registered env check 및 CLI audit 경로에서 env-missing-concrete 계산에 전달됩니다. config 없는 기본 경로도 빈 제외 목록으로 유지됩니다. 로컬 검증과 fresh-session pre-ship 리뷰가 통과했고, 게시 직전 확인한 원격 check에는 실패가 없었습니다.

Note: GitHub 정책상 PR 작성자 계정으로는 official approve를 제출할 수 없어 PR comment로 동일한 verdict를 남깁니다.

@JeremyDev87 JeremyDev87 self-assigned this May 6, 2026
@JeremyDev87 JeremyDev87 marked this pull request as ready for review May 6, 2026 14:15
@JeremyDev87 JeremyDev87 merged commit 274ab58 into master May 6, 2026
42 checks passed
@JeremyDev87 JeremyDev87 deleted the feature/104-env-contract-config branch May 6, 2026 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Maximus] CI/optional env contract allowlist config 추가

1 participant