From c0ae2f9283e0a1e0d606fb08a674c56abd4f8c0d Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Thu, 16 Apr 2026 14:10:52 +0000
Subject: [PATCH] docs: add error propagation note to createImportResolver
 JSDoc

Agent-Logs-Url: https://github.com/abhigyanpatwari/GitNexus/sessions/c3aec32d-2155-4808-88df-9cd6b2384174

Co-authored-by: magyargergo <11230420+magyargergo@users.noreply.github.com>
---
 .../src/core/ingestion/import-resolvers/resolver-factory.ts  | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/gitnexus/src/core/ingestion/import-resolvers/resolver-factory.ts b/gitnexus/src/core/ingestion/import-resolvers/resolver-factory.ts
index 4f00a559f..4caf748a7 100644
--- a/gitnexus/src/core/ingestion/import-resolvers/resolver-factory.ts
+++ b/gitnexus/src/core/ingestion/import-resolvers/resolver-factory.ts
@@ -17,6 +17,11 @@ import type { ImportResolverFn, ImportResolutionConfig } from './types.js';
  *
  * Chains strategies in declaration order — first non-null result wins.
  * Returns null only if every strategy returns null.
+ *
+ * Error behaviour: if a strategy throws, the error propagates immediately
+ * and remaining strategies are not tried.  Strategies are expected to be
+ * pure data transforms that never throw; any unexpected exception indicates
+ * a bug in the strategy implementation.
  */
 export function createImportResolver(config: ImportResolutionConfig): ImportResolverFn {
   const { strategies } = config;