Reduce cyclomatic complexity in raw-stream hotspots and consolidate utilities under src/utils

[Copilot]

Recent raw stream support increased complexity in core storage/event modules, creating maintainability hotspots (notably metadataUtil, EventStore, and partition/storage writers). This change set reduces branch-heavy logic where it adds value and centralizes reusable utilities under src/utils.

• Utility module consolidation

  • Moved utility files from src/*Util.js/src/util.js to src/utils/*.
  • Updated all internal imports, affected tests, package file inclusion, and publish-workflow packed-file expectations to the new paths.

• metadataUtil refinements

  • Kept low-complexity hot-path checks inline with Array.some(...) where dedicated helpers did not improve clarity.
  • Extracted binary JSON same-level matching/scanning logic into a dedicated reusable utility: src/utils/jsonUtil.js.
  • Preserved matcher behavior while simplifying object-branch handling in matcher tree construction.

• EventStore branch simplification

  • Extracted repeated predicate/raw argument normalization into a shared helper used by stream-access APIs.
  • Consolidated delegated hook-event handling (preCommit/preRead) into shared delegation methods to remove duplicated conditional branches.
  • Isolated query-type existence resolution into a dedicated method.

• WritableStorage / partition refactors

  • WritableStorage: extracted partition id/config/directory creation paths and streamlined secondary-index add paths to keep index.add(...) explicit without unnecessary delegation.
  • ReadablePartition: extracted iteration position resolution, reader selection, and header output assignment helpers.
  • WritablePartition: extracted write metadata/argument normalization and unbuffered-write threshold decision.

function normalizePredicateRaw(predicate, raw) {
  if (typeof predicate === 'boolean' && raw === false) {
    return { predicate: null, raw: predicate };
  }
  return { predicate, raw };
}

[albe]

@copilot We do not aim towards single digit complexity per function, but rather a low two digit value (20-30 is okayish) with occasional functions especially in the hot path being allowed to go higher. So avoid extracting methods from methods that are already low in complexity, unless it allows to extract logic into a generic util.

[albe]

Suggested change

	function isPlainObject(value) {
	return value !== null && typeof value === 'object' && !Array.isArray(value);
	}

[albe]

@copilot resolve the merge conflicts in this pull request

[Copilot]

@copilot resolve the merge conflicts in this pull request

Merge conflicts are resolved in commit 4aac3f9.

[Copilot]

@copilot resolve the merge conflicts in this pull request

Resolved in 0c5d557 by merging origin/main and fixing all resulting conflicts in the PR branch.

[coveralls]

coverage: 98.42% (-0.04%) from 98.459% — copilot/refactor-codebase-maintainability into main

[albe]

@copilot where is the -0.04% coverage coming from?

[Copilot]

@copilot where is the -0.04% coverage coming from?

It’s mainly from src/utils/metadataUtil.js: coverage there dropped from 99.67% on main to 98.78% in this branch due to 2 additional uncovered lines (5-7, the isPlainObject helper). There’s also 1 uncovered line in the new src/utils/jsonUtil.js path. Net effect is the small overall -0.04%.