Dev#13

Merged

Snider merged 29 commits intomainfrom

dev

Mar 24, 2026

Commits on Mar 24, 2026

feat: restore functional option pattern for New()

Snider
and
Virgil
committed
Merge pull request '[agent/codex] Review PR #28. Read CLAUDE.md first. Check: 1) API contract ...' (#29) from agent/review-pr--28--read-claude-md-first--che into dev
Virgil
committed
fix: address Codex review findings on PR #28

Snider
and
Virgil
committed
fix: prevent double IPC registration + empty service placeholder

Snider
and
Virgil
committed
fix: move HandleIPCEvents discovery to New() post-construction

Snider
and
Virgil
committed
feat: RegisterService with instance storage + interface discovery

Snider
and
Virgil
committed
feat: Options struct + Result methods + WithOption convenience

Snider
and
Virgil
committed
feat: App struct with New(Options) + Find() as method

Snider
and
Virgil
committed
fix: update Cli doc comment + tests for new Options contract

Snider
and
Virgil
committed
feat: Cli.New(c) constructor — Core uses it during construction

Snider
and
Virgil
committed
wip: checkpoint before v0.3.3 parity rewrite

Snider
and
Virgil
committed
feat: WithService with v0.3.3 name discovery + IPC handler auto-registration

Snider
and
Virgil
committed
feat: WithService with v0.3.3 name discovery + IPC handler auto-registration

Snider
and
Virgil
committed
feat: WithName for explicit service naming

Snider
and
Virgil
committed
test: lifecycle + HandleIPCEvents end-to-end via WithService

Snider
and
Virgil
committed
fix: WithServiceLock enables, New() applies after all opts — v0.3.3 parity

Snider
and
Virgil
committed
feat: MustServiceFor[T] + fix service names test for auto-registered cli

Snider
and
Virgil
committed
wip: v0.3.3 parity — Tasks 1-7 complete, data/embed tests need fixing

Snider
and
Virgil
committed
fix: Result.New handles (value, error) pairs correctly + embed test fixes

Snider
and
Virgil
committed
feat: New() constructors for Config, Fs + simplify contract.go init

Snider
and
Virgil
committed
fix: Service() returns instance, ServiceFor uses type assertion directly

Snider
and
Virgil
committed
feat: Core.Run() — ServiceStartup → Cli → ServiceShutdown lifecycle

Snider
and
Virgil
committed
feat: Core.Run() handles os.Exit on error

Snider
and
Virgil
committed
feat: New() returns *Core directly — no Result wrapper needed

Snider
and
Virgil
committed
Merge pull request 'feat: restore functional option pattern for New()' (#28) from feat/service-options into dev
Virgil
committed
fix: shutdown context, double IPC registration

Snider
and
Virgil
committed
Merge pull request 'fix: shutdown context + double IPC registration' (#36) from fix/codex-review-findings into dev
Virgil
committed
test: add _Bad/_Ugly tests + fix per-Core lock isolation

Snider
and
Virgil
committed
Merge pull request 'test: _Bad/_Ugly tests + per-Core lock isolation' (#37) from feat/test-coverage into dev
Virgil
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Dev#13

Dev#13
Snider merged 29 commits intomainfrom
dev

Commits on Mar 24, 2026

feat: restore functional option pattern for New()

Merge pull request '[agent/codex] Review PR #28. Read CLAUDE.md first. Check: 1) API contract ...' (#29) from agent/review-pr--28--read-claude-md-first--che into dev

fix: address Codex review findings on PR #28

fix: prevent double IPC registration + empty service placeholder

fix: move HandleIPCEvents discovery to New() post-construction

feat: RegisterService with instance storage + interface discovery

feat: Options struct + Result methods + WithOption convenience

feat: App struct with New(Options) + Find() as method

fix: update Cli doc comment + tests for new Options contract

feat: Cli.New(c) constructor — Core uses it during construction

wip: checkpoint before v0.3.3 parity rewrite

feat: WithService with v0.3.3 name discovery + IPC handler auto-registration

feat: WithService with v0.3.3 name discovery + IPC handler auto-registration

feat: WithName for explicit service naming

test: lifecycle + HandleIPCEvents end-to-end via WithService

fix: WithServiceLock enables, New() applies after all opts — v0.3.3 parity

feat: MustServiceFor[T] + fix service names test for auto-registered cli

wip: v0.3.3 parity — Tasks 1-7 complete, data/embed tests need fixing

fix: Result.New handles (value, error) pairs correctly + embed test fixes

feat: New() constructors for Config, Fs + simplify contract.go init

fix: Service() returns instance, ServiceFor uses type assertion directly

feat: Core.Run() — ServiceStartup → Cli → ServiceShutdown lifecycle

feat: Core.Run() handles os.Exit on error

feat: New() returns *Core directly — no Result wrapper needed

Merge pull request 'feat: restore functional option pattern for New()' (#28) from feat/service-options into dev

fix: shutdown context, double IPC registration

Merge pull request 'fix: shutdown context + double IPC registration' (#36) from fix/codex-review-findings into dev

test: add _Bad/_Ugly tests + fix per-Core lock isolation

Merge pull request 'test: _Bad/_Ugly tests + per-Core lock isolation' (#37) from feat/test-coverage into dev