skillrepo 4.7.0 → 4.8.1

package/src/test/commands/init-session-sync.test.mjs

@@ -0,0 +1,724 @@
+/**
+ * Unit tests for src/commands/init-session-sync.mjs (#894 / v3.1.2).
+ *
+ * Step 6 of `skillrepo init` owns the SessionStart-hook decision tree,
+ * the lowest-branch-coverage module in the CLI before these tests. The
+ * exported `installSessionSyncHook(...)` is directly unit-testable: it
+ * takes injected `deps` (`confirmFn`, `spawn`, `getCliVersion`) and a
+ * printer `p`, so we drive every branch WITHOUT running the full
+ * `runInit` orchestration.
+ *
+ * ## Decision tree under test
+ *
+ *   1. noSessionSync                       → OptedOut
+ *   2. !claudeTargeted                     → NotApplicable
+ *   3. non-npx, declined                   → Declined
+ *   4. non-npx, proceed                    → Installed/Updated/Unchanged
+ *   5. npx + existing global on PATH       → use-existing-global
+ *   6. npx + no global, declined           → Declined + manual steps
+ *   7. npx + no global, getCliVersion throw→ Skipped + manual steps
+ *   8. npx + no global, install fails      → Skipped
+ *   9. npx + no global, install succeeds   → Installed (auto-install)
+ *
+ * ## Environment control
+ *
+ * npx-mode is decided by `isTransientRunnerInvocation()`, which reads
+ * `process.env._` (suffix `/npx`) and `process.argv[1]` (cache
+ * substring). We force npx mode by setting `process.env._` to a value
+ * ending in `/npx`; non-npx mode by deleting `_` (the node:test
+ * runner's argv[1] contains no runner cache substring). We
+ * save/restore `_` and `PATH` in beforeEach/afterEach.
+ *
+ * Existing-global presence is decided by `resolveGlobalBinary()`,
+ * which scans `process.env.PATH`. A `skillrepo` shim on PATH makes it
+ * resolve; an empty/shim-free PATH makes it return null. HOME is
+ * isolated to a sandbox so any hook write lands in the sandbox.
+ */
+
+import { describe, it, beforeEach, afterEach } from "node:test";
+import assert from "node:assert/strict";
+import {
+  mkdtempSync,
+  mkdirSync,
+  rmSync,
+  readFileSync,
+  writeFileSync,
+  existsSync,
+} from "node:fs";
+import { join } from "node:path";
+import { tmpdir } from "node:os";
+
+import { installSessionSyncHook } from "../../commands/init-session-sync.mjs";
+import { SessionSyncAction } from "../../commands/session-sync-actions.mjs";
+import { buildHookCommand } from "../../lib/mergers/session-hook.mjs";
+import { SESSION_HOOK_FINGERPRINT } from "../../lib/artifact-registry.mjs";
+import {
+  captureHome,
+  setSandboxHome,
+  restoreHome,
+  assertHomeIsolated,
+} from "../helpers/sandbox-home.mjs";
+import { installShim, uninstallShim } from "../helpers/skillrepo-shim.mjs";
+
+let sandbox;
+let originalCwd;
+/** @type {import("../helpers/sandbox-home.mjs").HomeEnvSnapshot} */
+let originalHomeEnv;
+/** @type {string | undefined} */
+let originalUnderscore;
+/** @type {string | undefined} */
+let originalPath;
+
+/**
+ * Stub printer matching the `makePrinter` surface used by this module:
+ * only `success` and `warning` are called. Records every line so tests
+ * can assert on the user-facing copy (especially the manual-steps text
+ * emitted by `printManualSteps`).
+ */
+function makeStubPrinter() {
+  const success = [];
+  const warning = [];
+  return {
+    success(msg) {
+      success.push(msg);
+    },
+    warning(msg) {
+      warning.push(msg);
+    },
+    /** All lines, regardless of channel, joined for substring asserts. */
+    all() {
+      return [...success, ...warning].join("\n");
+    },
+    _success: success,
+    _warning: warning,
+  };
+}
+
+/** Force npx (transient-runner) mode via process.env._. */
+function forceNpxMode() {
+  process.env._ = "/usr/local/bin/npx";
+}
+
+/** Force non-npx mode by removing the `_` signal. */
+function forceNonNpxMode() {
+  delete process.env._;
+}
+
+/**
+ * Clear `skillrepo` off PATH entirely so `resolveGlobalBinary()` and
+ * `mergeSessionHook`'s resolver both return null. We point PATH at an
+ * empty sandbox bin dir (kept absolute so the locator doesn't drop it
+ * as a relative entry).
+ */
+function clearSkillrepoFromPath() {
+  const emptyBin = join(sandbox, "empty-bin");
+  mkdirSync(emptyBin, { recursive: true });
+  process.env.PATH = emptyBin;
+}
+
+function ASSERT_HOME_ISOLATED() {
+  assertHomeIsolated(tmpdir(), "init-session-sync tests");
+}
+
+function setup() {
+  sandbox = mkdtempSync(join(tmpdir(), "cli-init-session-sync-"));
+  mkdirSync(join(sandbox, "project"), { recursive: true });
+  mkdirSync(join(sandbox, "home"), { recursive: true });
+  originalCwd = process.cwd();
+  originalHomeEnv = captureHome();
+  originalUnderscore = process.env._;
+  originalPath = process.env.PATH;
+  process.chdir(join(sandbox, "project"));
+  setSandboxHome(join(sandbox, "home"));
+  ASSERT_HOME_ISOLATED();
+}
+
+function teardown() {
+  process.chdir(originalCwd);
+  // Restore PATH first (shim install/uninstall mutates it).
+  if (originalPath === undefined) delete process.env.PATH;
+  else process.env.PATH = originalPath;
+  if (originalUnderscore === undefined) delete process.env._;
+  else process.env._ = originalUnderscore;
+  restoreHome(originalHomeEnv);
+  if (sandbox) rmSync(sandbox, { recursive: true, force: true });
+}
+
+/** Absolute path to the project-local settings file in the sandbox. */
+function projectSettingsPath() {
+  return join(process.cwd(), ".claude", "settings.local.json");
+}
+
+/** Does the on-disk project settings file contain the SkillRepo hook? */
+function projectHasHook() {
+  const p = projectSettingsPath();
+  if (!existsSync(p)) return false;
+  const parsed = JSON.parse(readFileSync(p, "utf-8"));
+  return (parsed.hooks?.SessionStart ?? [])
+    .flatMap((g) => g?.hooks ?? [])
+    .some((h) => h?.command?.includes(SESSION_HOOK_FINGERPRINT));
+}
+
+// ──────────────────────────────────────────────────────────────────
+// 1. noSessionSync → OptedOut
+// ──────────────────────────────────────────────────────────────────
+
+describe("init-session-sync — opted out / not applicable", () => {
+  beforeEach(setup);
+  afterEach(teardown);
+
+  it("returns OptedOut and writes nothing when --no-session-sync", async () => {
+    const p = makeStubPrinter();
+    const r = await installSessionSyncHook({
+      noSessionSync: true,
+      claudeTargeted: true,
+      yes: true,
+      json: false,
+      global: false,
+      p,
+    });
+
+    assert.equal(r.action, SessionSyncAction.OptedOut);
+    assert.equal(r.path, null);
+    assert.equal(r.globalInstallActive, false);
+    assert.match(p.all(), /--no-session-sync/);
+    assert.equal(projectHasHook(), false, "must not write a hook");
+  });
+
+  it("returns NotApplicable when Claude Code is not a target", async () => {
+    const p = makeStubPrinter();
+    const r = await installSessionSyncHook({
+      noSessionSync: false,
+      claudeTargeted: false,
+      yes: true,
+      json: false,
+      global: false,
+      p,
+    });
+
+    assert.equal(r.action, SessionSyncAction.NotApplicable);
+    assert.equal(r.path, null);
+    assert.equal(r.globalInstallActive, false);
+    assert.match(p.all(), /Claude Code-specific/);
+    assert.equal(projectHasHook(), false);
+  });
+});
+
+// ──────────────────────────────────────────────────────────────────
+// 3 + 4. Non-npx branch
+// ──────────────────────────────────────────────────────────────────
+
+describe("init-session-sync — non-npx branch", () => {
+  let shimHandle;
+
+  beforeEach(() => {
+    setup();
+    forceNonNpxMode();
+  });
+  afterEach(() => {
+    uninstallShim(shimHandle);
+    shimHandle = undefined;
+    teardown();
+  });
+
+  it("declined (prompt says no) → Declined, no write", async () => {
+    // No shim needed: we never reach the merge step.
+    clearSkillrepoFromPath();
+    const p = makeStubPrinter();
+    const r = await installSessionSyncHook({
+      noSessionSync: false,
+      claudeTargeted: true,
+      yes: false,
+      json: false,
+      global: false,
+      p,
+      deps: { confirmFn: () => false },
+    });
+
+    assert.equal(r.action, SessionSyncAction.Declined);
+    assert.equal(r.path, null);
+    assert.equal(r.globalInstallActive, false);
+    assert.match(p.all(), /session-sync enable/);
+    assert.equal(projectHasHook(), false);
+  });
+
+  it("proceed via --yes → Installed (fresh hook on disk)", async () => {
+    // Shim on PATH so mergeSessionHook's resolver finds a binary.
+    shimHandle = installShim(join(sandbox, "home"));
+    const p = makeStubPrinter();
+    const r = await installSessionSyncHook({
+      noSessionSync: false,
+      claudeTargeted: true,
+      yes: true,
+      json: false,
+      global: false,
+      p,
+    });
+
+    assert.equal(r.action, SessionSyncAction.Installed);
+    assert.equal(r.path, ".claude/settings.local.json");
+    assert.equal(r.globalInstallActive, true);
+    assert.match(p.all(), /installed/i);
+    assert.equal(projectHasHook(), true);
+  });
+
+  it("proceed via confirm prompt (returns true) → Installed", async () => {
+    // Exercises the confirmFn-returns-true path (yes:false but prompt
+    // accepts), distinct from the --yes short-circuit above.
+    shimHandle = installShim(join(sandbox, "home"));
+    const p = makeStubPrinter();
+    const r = await installSessionSyncHook({
+      noSessionSync: false,
+      claudeTargeted: true,
+      yes: false,
+      json: false,
+      global: false,
+      p,
+      deps: { confirmFn: () => true },
+    });
+
+    assert.equal(r.action, SessionSyncAction.Installed);
+    assert.equal(r.globalInstallActive, true);
+    assert.equal(projectHasHook(), true);
+  });
+
+  it("re-run with same binary → Unchanged (idempotent)", async () => {
+    shimHandle = installShim(join(sandbox, "home"));
+    const p1 = makeStubPrinter();
+    await installSessionSyncHook({
+      noSessionSync: false,
+      claudeTargeted: true,
+      yes: true,
+      json: false,
+      global: false,
+      p: p1,
+    });
+
+    const p2 = makeStubPrinter();
+    const r = await installSessionSyncHook({
+      noSessionSync: false,
+      claudeTargeted: true,
+      yes: true,
+      json: false,
+      global: false,
+      p: p2,
+    });
+
+    assert.equal(r.action, SessionSyncAction.Unchanged);
+    assert.equal(r.globalInstallActive, true);
+    assert.match(p2.all(), /already installed/i);
+  });
+
+  it("existing stale SkillRepo hook → Updated", async () => {
+    shimHandle = installShim(join(sandbox, "home"));
+    // Seed a SkillRepo hook pointing at a different binary path so the
+    // command differs and triggers the in-place 'updated' replacement.
+    mkdirSync(join(process.cwd(), ".claude"), { recursive: true });
+    const stale = buildHookCommand("/old/location/skillrepo");
+    writeFileSync(
+      projectSettingsPath(),
+      JSON.stringify(
+        {
+          hooks: {
+            SessionStart: [{ hooks: [{ type: "command", command: stale }] }],
+          },
+        },
+        null,
+        2,
+      ),
+    );
+
+    const p = makeStubPrinter();
+    const r = await installSessionSyncHook({
+      noSessionSync: false,
+      claudeTargeted: true,
+      yes: true,
+      json: false,
+      global: false,
+      p,
+    });
+
+    assert.equal(r.action, SessionSyncAction.Updated);
+    assert.equal(r.globalInstallActive, true);
+    assert.match(p.all(), /updated/i);
+  });
+
+  it("Skipped when no binary resolvable (bare-install PATH miss)", async () => {
+    // Non-npx, proceed, but skillrepo is NOT on PATH → mergeSessionHook
+    // returns action 'skipped' with an actionable reason. Exercises the
+    // Skipped branch of tryMergeAndPrint (252-281) and isHookActive's
+    // false arm for a non-active action.
+    clearSkillrepoFromPath();
+    const p = makeStubPrinter();
+    const r = await installSessionSyncHook({
+      noSessionSync: false,
+      claudeTargeted: true,
+      yes: true,
+      json: false,
+      global: false,
+      p,
+    });
+
+    assert.equal(r.action, SessionSyncAction.Skipped);
+    assert.equal(r.globalInstallActive, false);
+    assert.match(p.all(), /skillrepo/i);
+    assert.equal(projectHasHook(), false);
+  });
+
+  it("Failed when settings file is corrupt (disk error caught)", async () => {
+    // A present-but-corrupt settings file makes mergeSessionHook throw;
+    // tryMergeAndPrint's catch (272-280) converts it to Failed and the
+    // module does NOT propagate the throw.
+    shimHandle = installShim(join(sandbox, "home"));
+    mkdirSync(join(process.cwd(), ".claude"), { recursive: true });
+    writeFileSync(projectSettingsPath(), "{ not valid json");
+
+    const p = makeStubPrinter();
+    const r = await installSessionSyncHook({
+      noSessionSync: false,
+      claudeTargeted: true,
+      yes: true,
+      json: false,
+      global: false,
+      p,
+    });
+
+    assert.equal(r.action, SessionSyncAction.Failed);
+    assert.equal(r.path, null);
+    assert.equal(r.globalInstallActive, false);
+    assert.match(p.all(), /Session sync failed/i);
+  });
+});
+
+// ──────────────────────────────────────────────────────────────────
+// 5. npx + existing global on PATH
+// ──────────────────────────────────────────────────────────────────
+
+describe("init-session-sync — npx with existing global", () => {
+  let shimHandle;
+
+  beforeEach(() => {
+    setup();
+    forceNpxMode();
+    // A resolvable global skillrepo on PATH → resolveGlobalBinary()
+    // returns a path → use-existing-global branch.
+    shimHandle = installShim(join(sandbox, "home"));
+  });
+  afterEach(() => {
+    uninstallShim(shimHandle);
+    shimHandle = undefined;
+    teardown();
+  });
+
+  it("uses the existing global and installs the hook → Installed", async () => {
+    const p = makeStubPrinter();
+    const r = await installSessionSyncHook({
+      noSessionSync: false,
+      claudeTargeted: true,
+      yes: true,
+      json: false,
+      global: false,
+      p,
+    });
+
+    assert.equal(r.action, SessionSyncAction.Installed);
+    // globalInstallActive is forced true regardless of hook outcome.
+    assert.equal(r.globalInstallActive, true);
+    assert.match(p.all(), /Found global skillrepo/);
+    assert.match(p.all(), /using it for session sync/);
+    assert.equal(projectHasHook(), true);
+  });
+
+  it("forces globalInstallActive=true even when the hook is Unchanged", async () => {
+    // Second run: hook is already present → mergeSessionHook returns
+    // Unchanged, but the existing-global branch still overrides
+    // globalInstallActive to true (line 177-178). isHookActive would
+    // already be true here, so to truly prove the override we also
+    // cover the Skipped-but-still-active case below.
+    const p1 = makeStubPrinter();
+    await installSessionSyncHook({
+      noSessionSync: false,
+      claudeTargeted: true,
+      yes: true,
+      json: false,
+      global: false,
+      p: p1,
+    });
+
+    const p2 = makeStubPrinter();
+    const r = await installSessionSyncHook({
+      noSessionSync: false,
+      claudeTargeted: true,
+      yes: true,
+      json: false,
+      global: false,
+      p: p2,
+    });
+
+    assert.equal(r.action, SessionSyncAction.Unchanged);
+    assert.equal(r.globalInstallActive, true);
+  });
+});
+
+// ──────────────────────────────────────────────────────────────────
+// 6-9. npx + no global (auto-install branch)
+// ──────────────────────────────────────────────────────────────────
+
+describe("init-session-sync — npx without global (auto-install)", () => {
+  let shimHandle;
+
+  beforeEach(() => {
+    setup();
+    forceNpxMode();
+    // No skillrepo on PATH → resolveGlobalBinary() (line 132) returns
+    // null → auto-install branch.
+    clearSkillrepoFromPath();
+  });
+  afterEach(() => {
+    uninstallShim(shimHandle);
+    shimHandle = undefined;
+    teardown();
+  });
+
+  it("declined → Declined + manual-steps hint (npx command)", async () => {
+    const p = makeStubPrinter();
+    const r = await installSessionSyncHook({
+      noSessionSync: false,
+      claudeTargeted: true,
+      yes: false,
+      json: false,
+      global: false,
+      p,
+      deps: { confirmFn: () => false },
+    });
+
+    assert.equal(r.action, SessionSyncAction.Declined);
+    assert.equal(r.globalInstallActive, false);
+    assert.match(p.all(), /declined/i);
+    // printManualSteps uses the detected runner's install command.
+    // Under forced npx mode the runner resolves to "npx" → the
+    // canonical "npm install -g skillrepo" command.
+    assert.match(p.all(), /npm install -g skillrepo/);
+    assert.match(p.all(), /session-sync enable/);
+    assert.equal(projectHasHook(), false);
+  });
+
+  it("getCliVersion throws → Skipped + manual steps", async () => {
+    const p = makeStubPrinter();
+    const r = await installSessionSyncHook({
+      noSessionSync: false,
+      claudeTargeted: true,
+      yes: true, // proceed without prompting
+      json: false,
+      global: false,
+      p,
+      deps: {
+        getCliVersion: () => {
+          throw new Error("boom");
+        },
+        // spawn must never be reached; fail loudly if it is.
+        spawn: () => {
+          throw new Error("spawn must not run after version-read failure");
+        },
+      },
+    });
+
+    assert.equal(r.action, SessionSyncAction.Skipped);
+    assert.equal(r.globalInstallActive, false);
+    assert.match(p.all(), /Could not determine CLI version/);
+    assert.match(p.all(), /boom/);
+    assert.match(p.all(), /npm install -g skillrepo/);
+    assert.equal(projectHasHook(), false);
+  });
+
+  it("install fails (npm non-zero) → Skipped", async () => {
+    // Stub spawn returns a child that closes with a non-zero exit code,
+    // so installSkillrepoGlobally reports success:false (npm-nonzero).
+    const p = makeStubPrinter();
+    const r = await installSessionSyncHook({
+      noSessionSync: false,
+      claudeTargeted: true,
+      yes: true,
+      json: false,
+      global: false,
+      p,
+      deps: {
+        getCliVersion: () => "9.9.9",
+        spawn: makeSpawnStub({ exitCode: 1 }),
+      },
+    });
+
+    assert.equal(r.action, SessionSyncAction.Skipped);
+    assert.equal(r.globalInstallActive, false);
+    assert.match(p.all(), /Could not install skillrepo globally/);
+    assert.match(p.all(), /npm install -g skillrepo/);
+    assert.equal(projectHasHook(), false);
+  });
+
+  it("install succeeds → Installed (auto-install path)", async () => {
+    // Stub spawn closes with code 0 AND, just before emitting close,
+    // installs a real shim on PATH so installSkillrepoGlobally's
+    // post-install resolveGlobalBinary() finds the binary. The
+    // top-level resolveGlobalBinary() at line 132 already ran (and
+    // returned null) before the spawn, so we still took the
+    // auto-install branch.
+    let installedShim;
+    const p = makeStubPrinter();
+    const r = await installSessionSyncHook({
+      noSessionSync: false,
+      claudeTargeted: true,
+      yes: true,
+      json: false,
+      global: false,
+      p,
+      deps: {
+        getCliVersion: () => "9.9.9",
+        spawn: makeSpawnStub({
+          exitCode: 0,
+          onBeforeClose: () => {
+            installedShim = installShim(join(sandbox, "home"));
+          },
+        }),
+      },
+    });
+
+    // Clean up the shim PATH mutation done inside the spawn stub.
+    shimHandle = installedShim;
+
+    assert.equal(r.action, SessionSyncAction.Installed);
+    assert.equal(r.globalInstallActive, true);
+    assert.match(p.all(), /Running: npm install -g skillrepo@9\.9\.9/);
+    assert.match(p.all(), /Installed skillrepo@9\.9\.9 globally/);
+    assert.equal(projectHasHook(), true);
+  });
+
+  it("falls back to the real getCliVersion when none is injected", async () => {
+    // deps.getCliVersion omitted → the `?? getCliVersion` default
+    // (line 203) is exercised. The real read returns the package's
+    // own version; spawn is still stubbed so no npm runs. Asserting
+    // the printed version is a non-empty semver-ish string proves the
+    // real reader ran.
+    let installedShim;
+    const p = makeStubPrinter();
+    const r = await installSessionSyncHook({
+      noSessionSync: false,
+      claudeTargeted: true,
+      yes: true,
+      json: false,
+      global: false,
+      p,
+      deps: {
+        spawn: makeSpawnStub({
+          exitCode: 0,
+          onBeforeClose: () => {
+            installedShim = installShim(join(sandbox, "home"));
+          },
+        }),
+      },
+    });
+
+    shimHandle = installedShim;
+
+    assert.equal(r.action, SessionSyncAction.Installed);
+    assert.equal(r.globalInstallActive, true);
+    assert.match(p.all(), /Running: npm install -g skillrepo@\d+\.\d+\.\d+/);
+  });
+
+  it("non-Error thrown by getCliVersion → String(err) in the message", async () => {
+    // Throwing a non-Error (a bare string) makes `err?.message`
+    // undefined, exercising the `?? String(err)` arm at line 209.
+    const p = makeStubPrinter();
+    const r = await installSessionSyncHook({
+      noSessionSync: false,
+      claudeTargeted: true,
+      yes: true,
+      json: false,
+      global: false,
+      p,
+      deps: {
+        getCliVersion: () => {
+          throw "string-failure";
+        },
+      },
+    });
+
+    assert.equal(r.action, SessionSyncAction.Skipped);
+    assert.match(
+      p.all(),
+      /Could not determine CLI version for global install: string-failure/,
+    );
+  });
+
+  it("--json mode runs install silently and still installs the hook", async () => {
+    // json:true → installSkillrepoGlobally gets outputMode 'silent',
+    // which pipes stdout/stderr. The spawn stub provides those streams.
+    let installedShim;
+    const p = makeStubPrinter();
+    const r = await installSessionSyncHook({
+      noSessionSync: false,
+      claudeTargeted: true,
+      yes: true,
+      json: true,
+      global: false,
+      p,
+      deps: {
+        getCliVersion: () => "9.9.9",
+        spawn: makeSpawnStub({
+          exitCode: 0,
+          silent: true,
+          onBeforeClose: () => {
+            installedShim = installShim(join(sandbox, "home"));
+          },
+        }),
+      },
+    });
+
+    shimHandle = installedShim;
+
+    assert.equal(r.action, SessionSyncAction.Installed);
+    assert.equal(r.globalInstallActive, true);
+    assert.equal(projectHasHook(), true);
+  });
+});
+
+// ──────────────────────────────────────────────────────────────────
+// Spawn stub for installSkillrepoGlobally
+// ──────────────────────────────────────────────────────────────────
+
+import { EventEmitter } from "node:events";
+import { Readable } from "node:stream";
+
+/**
+ * Build an injectable `spawn` stub matching the contract
+ * `installSkillrepoGlobally` expects:
+ *   - returns a child EventEmitter
+ *   - emits `close` with an exit code on a later tick
+ *   - in `silent` mode the child carries readable `stdout`/`stderr`
+ *
+ * @param {object} opts
+ * @param {number} opts.exitCode - exit code to emit via `close`.
+ * @param {boolean} [opts.silent] - attach stdout/stderr streams (for
+ *        the `outputMode: "silent"` capture path).
+ * @param {Function} [opts.onBeforeClose] - run synchronously right
+ *        before the `close` event fires (used to install a PATH shim
+ *        so the post-install binary resolution succeeds).
+ */
+function makeSpawnStub({ exitCode, silent = false, onBeforeClose }) {
+  return function spawnStub() {
+    const child = new EventEmitter();
+    if (silent) {
+      // Provide empty readable streams; installSkillrepoGlobally
+      // attaches `data` listeners to drain/capture them.
+      child.stdout = Readable.from([]);
+      child.stderr = Readable.from([]);
+    }
+    child.kill = () => {};
+    // Fire close on the next tick so the awaiting Promise's listeners
+    // are registered first.
+    setImmediate(() => {
+      if (onBeforeClose) onBeforeClose();
+      child.emit("close", exitCode);
+    });
+    return child;
+  };
+}