Skip to content

refactor: consolidate unique helpers - #31

Merged
GRAMMAC1 merged 1 commit into
masterfrom
codex-consolidate-unique-utils
Jul 7, 2026
Merged

refactor: consolidate unique helpers#31
GRAMMAC1 merged 1 commit into
masterfrom
codex-consolidate-unique-utils

Conversation

@GRAMMAC1

@GRAMMAC1 GRAMMAC1 commented Jul 7, 2026

Copy link
Copy Markdown
Owner

Summary

  • Centralize generic unique in core text utilities.
  • Add uniqueNonEmptyStrings for existing call sites that relied on empty-string filtering.
  • Replace duplicate core parser and aggregate implementations with shared utilities.
  • Reuse the core non-empty string helper from MCP tool utilities while preserving the existing MCP unique export.

Validation

  • pnpm verify
  • pre-commit Biome check during commit

Note: sandboxed pnpm verify passed check/lint/typecheck/tests but failed at the harness step because tsx could not create its IPC pipe under the sandbox. The non-sandbox rerun passed.

@GRAMMAC1
GRAMMAC1 merged commit 9a858f6 into master Jul 7, 2026
1 check passed
@GRAMMAC1
GRAMMAC1 deleted the codex-consolidate-unique-utils branch July 7, 2026 05:02
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