refactor: unify duplicate Logger interfaces into single canonical type

[yuanrengu]

Summary

23 files had local Logger/StoreLogger/PluginLogger/PipelineLogger/RunnerLogger/ExtractorLogger/TriggerLogger interface definitions with identical shapes ({debug?, info, warn, error}). This PR replaces all duplicates with imports from the canonical Logger in src/core/types.ts.

Changes

• 17 files: local interface Logger removed → import type { Logger } from ../types.js
• 6 files: named variants replaced with type aliases for backward compatibility:
  • StoreLogger → type StoreLogger = Logger (exported, used by tcvdb-client/factory)
  • PluginLogger → type PluginLogger = Logger (exported, used by offload/*)
  • PipelineLogger → type PipelineLogger = Logger (exported, used by seed-runtime)
  • RunnerLogger → type RunnerLogger = Logger
  • ExtractorLogger → type ExtractorLogger = Logger
  • TriggerLogger → type TriggerLogger = Logger

Preserved as-is (different contract)

• CheckpointLogger in checkpoint.ts — only info + optional warn (no error)
• Logger in ensure-hook-policy.ts — only info, warn, debug? (no error)

Impact

• -128 lines (35 insertions, 163 deletions)
• Zero behavioral change — type-only refactor
• All named variants preserved as type aliases for backward compatibility
• Build verified: npx tsdown passes (714.61 kB, no errors)
• No circular dependencies: core/types.ts has zero imports

Test plan

• npx tsdown builds successfully
• No circular dependency chains
• All import type paths verified correct (relative to each file)
• Named type aliases (StoreLogger, PluginLogger, etc.) remain importable from original modules
• No runtime behavior changes (type-only refactor)

[Maxwell-Code07]

Hi @yuanrengu, thanks for the PR! 🧹

Unifying duplicate Logger interfaces across the codebase is a worthwhile cleanup direction. We'll review the changes and get back to you shortly.

Thanks for the contribution! 🙏

[yuanrengu]

@Maxwell-Code07 Thanks! I've been diving into agent memory systems lately and learning a lot through this project. Happy to help improve it — let me know if any changes are needed.

[withRiver]

LGTM. This PR will be merged soon, thanks for your contribution!

There is only one point that needs attention. The comment for interface Logger in src/core/type.ts(Line 17-22) is now outdated after this PR — the relationship has been inverted. StoreLogger/RunnerLogger are now aliases of Logger, not the other way around. Suggest updating to:

/**
 * Canonical logger interface used across all TDAI modules.
 *
 * Named variants (StoreLogger, PluginLogger, etc.) are type aliases
 * of this interface, kept for backward compatibility.
 */
 export interface Logger {

[withRiver]

LGTM. This PR will be merged soon, thanks for your contribution!

There is only one point that needs attention. The comment for interface Logger in src/core/type.ts(Line 17-22) is now outdated after this PR — the relationship has been inverted. StoreLogger/RunnerLogger are now aliases of Logger, not the other way around. Suggest updating to:

/**
 * Canonical logger interface used across all TDAI modules.
 *
 * Named variants (StoreLogger, PluginLogger, etc.) are type aliases
 * of this interface, kept for backward compatibility.
 */
 export interface Logger {

@yuanrengu Hello, would it be convenient for you to update the comment according to the suggestions mentioned above? After updating the comment, this PR can be merged.

[yuanrengu]

LGTM. This PR will be merged soon, thanks for your contribution!
There is only one point that needs attention. The comment for interface Logger in src/core/type.ts(Line 17-22) is now outdated after this PR — the relationship has been inverted. StoreLogger/RunnerLogger are now aliases of Logger, not the other way around. Suggest updating to:

/**
 * Canonical logger interface used across all TDAI modules.
 *
 * Named variants (StoreLogger, PluginLogger, etc.) are type aliases
 * of this interface, kept for backward compatibility.
 */
 export interface Logger {

@yuanrengu Hello, would it be convenient for you to update the comment according to the suggestions mentioned above? After updating the comment, this PR can be merged.

@withRiver Thanks for the review! Updated the JSDoc comment on Logger in src/core/types.ts to clarify that StoreLogger/PluginLogger etc. are now type aliases of the canonical Logger interface. Please take another look, thanks!