Skip to content

[R-101] review-only - #163

Open
yzsnstotz wants to merge 1 commit into
mainfrom
uld-mvp-2026-08-06/R-101
Open

yzsnstotz wants to merge 1 commit into
mainfrom
uld-mvp-2026-08-06/R-101

Conversation

@yzsnstotz

Copy link
Copy Markdown
Owner

Changes

  • Memoize Meridian configuration on first property access while preserving the existing config export contract.
  • Keep caller identity and model catalog initialization off configuration-free CLI paths.
  • Add package.json-backed --version output and advertise it in --help.
  • Add empty-environment positive and negative regression coverage.

Deletions Claimed

None.

Validation

  • Node 22 empty-environment identity regression tests: 4 passed.
  • Existing CLI and configuration focused suites: 43 passed.
  • npm run typecheck
  • focused ESLint on all touched TypeScript files
  • npm run build
  • Compiled env -i probes: --version exit 0, --help exit 0, hub entry exit 1 with TELEGRAM_BOT_TOKEN and ALLOWED_USER_IDS named.
  • Full suite has four baseline-identical host-sensitive failures, reproduced on untouched origin/main: monitor logger exit race plus three CLI resolver assertions that discover /opt/homebrew/bin/codex.

Review-only; not merged. No running Meridian process was restarted, stopped, or hot-updated.

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