feat(zunit): modernize core — native config, namespaced internals, signal-based timeout

[ss-o]

Summary

• Native Zsh config (.zunit.zsh) replaces .zunit.yml as the primary configuration format, using plain shell variables (ZUNIT_TESTS_DIR, ZUNIT_TIME_LIMIT, etc.); legacy .zunit.yml is preserved as a fallback for backward compatibility
• Full _zunit_* namespace for all internal symbols; public test helpers (color, load, assert, pass, fail, error, skip) kept as thin wrappers — existing test files require no changes
• Bundled revolver fully namespaced to _zunit_revolver_* (eliminates dependency on an external revolver binary); statefile path fixed from $PPID to $$ for correct process isolation
• Signal-based async timeout replaces the while kill -0 busy-wait polling loop with an ALRM trap + background timer subprocess
• Anonymous function test isolation wraps each test body in () { ... } to prevent local variable leakage between tests
• Z-Shell plugin standard compliance: ZERO handling, Plugins hash, and fpath guard added to src/zunit.zsh
• Echo correctness: removed spurious leading ( from output calls across run/init/events
• Repo config migrated: .zunit.zsh added for this repository

Deferred to a follow-up PR: zsh/zpty-based async runner redesign (spec §3) — independent higher-risk change.

Test plan

• All 111 existing tests pass (./zunit run tests/)
• zunit init now generates .zunit.zsh (verified by updated cli.zunit test)
• .zunit.yml fallback still recognised (config load order in src/zunit.zsh)
• Revolver spinner renders correctly (no external revolver binary needed)
• zsh build.zsh produces a clean binary

🤖 Generated with Claude Code

[Copilot]

Pull request overview

This PR modernizes ZUnit’s core by switching to a native Zsh-based config format (.zunit.zsh), fully namespacing internal symbols (including bundling a namespaced revolver spinner), and replacing polling-based test timeouts with a signal-based approach—while keeping existing test helper APIs compatible via thin wrappers.

Changes:

• Add native .zunit.zsh config loading (with .zunit.yml fallback) and migrate this repo to the new config.
• Namespace internal helpers (_zunit_*) and bundle a namespaced revolver (_zunit_revolver_*) to remove the external dependency.
• Replace busy-wait timeouts with an ALRM-trap + background timer and add anonymous-function wrapping around test bodies.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
tests/cli.zunit	Updates CLI tests for .zunit.zsh messaging and removes the external revolver dependency check.
src/zunit.zsh	Loads .zunit.zsh config first, adds plugin-standard ZERO/Plugins/fpath handling, and checks for internal revolver.
src/helpers.zsh	Keeps public helpers as wrappers while moving implementations into _zunit_* internals.
src/events.zsh	Switches output styling and spinner control calls to namespaced internals.
src/commands/run.zsh	Fixes output formatting, uses namespaced spinner, adds test-body isolation, and implements signal-based timeouts.
src/commands/init.zsh	Generates .zunit.zsh instead of .zunit.yml and updates init examples.
lib/revolver.zsh	Namespaces revolver functions and changes statefile keying from $PPID to $$.
lib/color.zsh	Removes implicit execution so it behaves purely as a library in the built artifact.
.zunit.zsh	Adds the repo’s new native ZUnit configuration.

Comments suppressed due to low confidence (1)

src/commands/run.zsh:556

• This comment still mentions .zunit.yml, but config is now primarily loaded from .zunit.zsh (with .zunit.yml as fallback). Update the comment to reflect the new config filename or refer to “config” generically.

  [[ -z $tap ]] && _zunit_revolver start 'Loading tests'

  # If no arguments are passed, try to work out where the tests are
  if [[ ${#arguments} -eq 0 ]]; then
    # Check for a path defined in .zunit.yml

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.