Impossible to bundle a circular schema after dereferencing it

[paztis]

If a schema has circular ancestor, it is possible to dereference it but if we want to re-reference it (for stringification for example), it is impossible to bundle it.

Tested in the unit tests with specs/circular/circular-indirect-ancestor.yaml

let parser = new $RefParser();
const schemaDeref = await parser.dereference(path.rel("specs/circular/circular-indirect-ancestor.yaml"));

const schema = await parser.bundle(schemaDeref);

This crashed with Maximum call stack in 2 places:

• in resolve-externals.js crawl function easily fixable with this code:

  function crawl (obj, path, $refs, options, ancestors = []) {
  let promises = [];
  
  if (obj && typeof obj === "object" && !ArrayBuffer.isView(obj)) {
  if ($Ref.isExternal$Ref(obj)) {
    promises.push(resolve$Ref(obj, path, $refs, options));
  }
  else if (ancestors.indexOf(obj) === -1) {
    const nextAncestors = [...ancestors, obj];
    for (let key of Object.keys(obj)) {
      let keyPath = Pointer.join(path, key);
      let value = obj[key];
  
      if ($Ref.isExternal$Ref(value)) {
        promises.push(resolve$Ref(value, keyPath, $refs, options));
      }
      else {
        promises = promises.concat(crawl(value, keyPath, $refs, options, nextAncestors));
      }
    }
  }
  }
  
  return promises;
  }

• then in bundle.js in crawl function here I don't know how to create new references in case of circular detection

[paztis]

I've create a PR with a new method rereference that exactly do this job in sync: #221

Cab you look at it ?

[paztis]

Hello

Any new about it ? Have you found any time to look at the PR ?

[philsturgeon]

Why would you dereference AND bundle? I don't understand the use case so its hard to imagine a solution.

[paztis]

My ui is based over schemas. I need to use schemas everywhere, and don't want to support reference in tons of places.

But my ui need also to save in session storage the schemas. And I cannot serialize the without re-referencing them

Le jeu. 6 mai 2021 à 13:52, Phil Sturgeon @.***> a écrit :

Why would you dereference AND bundle? I don't understand the use case so its hard to imagine a solution.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/APIDevTools/json-schema-ref-parser/issues/219#issuecomment-833462253, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABIKMMLCZK3WGMIQMKANKDTTMJ7JPANCNFSM4Y65A47A .

[philsturgeon]

@paztis can you see if this works in 9.0.9? We've done a lot of work on making various circular scenarios work better.