Tools: 🐛 × Expected relative file path but got:

Created on 9 Aug 2020  ·  9Comments  ·  Source: rome/tools


Environment

PS C:\Users\landi\code\typing-poc> yarn -s rome rage --summary
Environment Variables
  Object {
    COLORTERM: "truecolor"
    LANG: "en_US.UTF-8"
    PATH: "C:\\Users\\landi\\AppData\\Local\\Temp\\yarn--1596923824739-0.7116522805989045;C:\\Users\\landi\\code\\typing-poc\\node_modules\\.bin;C:\\Users\\landi\\AppData\\Local\\Yarn\\Data\\link\    
    \node_modules\\.bin;C:\\Program Files\\libexec\\lib\\node_modules\\npm\\bin\\node-gyp-bin;C:\\Program Files\\lib\\node_modules\\npm\\bin\\node-gyp-bin;C:\\Program
    Files\\nodejs\\node_modules\\npm\\bin\\node-gyp-bin;C:\\Python38\\Scripts\\;C:\\Python38\\;C:\\Program Files (x86)\\Common
    Files\\Oracle\\Java\\javapath;C:\\Windows\\system32;C:\\Windows;C:\\Windows\\System32\\Wbem;C:\\Windows\\System32\\WindowsPowerShell\\v1.0\\;C:\\Windows\\System32\\OpenSSH\\;C:\\Program Files     
    (x86)\\NVIDIA Corporation\\PhysX\\Common;C:\\ProgramData\\chocolatey\\bin;C:\\Program Files\\dotnet\\;C:\\Program Files\\Microsoft SQL Server\\130\\Tools\\Binn\\;C:\\Program Files\\NVIDIA
    Corporation\\NVIDIA NvDLISR;C:\\WINDOWS\\system32;C:\\WINDOWS;C:\\WINDOWS\\System32\\Wbem;C:\\WINDOWS\\System32\\WindowsPowerShell\\v1.0\\;C:\\WINDOWS\\System32\\OpenSSH\\;C:\\Program
    Files\\Microsoft VS Code\\bin;C:\\ProgramData\\chocolatey\\lib\\Elixir/bin;C:\\Program
    Files\\postgresql\\10\\bin;C:\\WINDOWS\\system32;C:\\WINDOWS;C:\\WINDOWS\\System32\\Wbem;C:\\WINDOWS\\System32\\WindowsPowerShell\\v1.0\\;C:\\WINDOWS\\System32\\OpenSSH\\;C:\\Program
    Files\\Docker\\Docker\\resources\\bin;C:\\ProgramData\\DockerDesktop\\version-bin;C:\\ProgramData\\chocolatey\\lib\\Elixir/bin;C:\\Program Files\\Git\\cmd;C:\\Go\\bin;C:\\Program
    Files\\nodejs\\;C:\\Program Files (x86)\\Yarn\\bin\\;C:\\Users\\landi\\AppData\\Local\\Microsoft\\WindowsApps;C:\\Program Files (x86)\\Atmel\\Flip
    3.4.7\\bin;C:\\Users\\landi\\AppData\\Local\\hyper\\app-3.0.2\\resources\\bin;C:\\Users\\landi\\go\\bin;C:\\Users\\landi\\.dotnet\\tools;C:\\Users\\landi\\AppData\\Local\\Microsoft\\WindowsApp    
    s;C:\\Users\\landi\\go\\bin;C:\\Users\\landi\\AppData\\Roaming\\npm;C:\\Users\\landi\\AppData\\Local\\Yarn\\bin"
    TERM_PROGRAM: "vscode"
  }

User Config
  unset

Rome Version
  10.0.3-beta

Node Version
  14.4.0

Platform
  win32 x64 10.0.19041

Terminal Features
  Object {
    background: "unknown"
    colorDepth: 24
    columns: 200
    cursor: true
    hyperlinks: true
    isTTY: true
    unicode: false
  }

Client Flags
  Object {
    clientName: "cli"
    cwd: "C:\\Users\\landi\\code\\typing-poc"
    realCwd: "C:\\Users\\landi\\code\\typing-poc"
    silent: false
  }

Server Status
  Object {
    projects: Array [
      Object {id: 0}
      Object {id: 1}
    ]
    server: Object {
      heapTotal: 26_079_232
      pid: 1_636
      uptime: 0.1872217
    }
    workers: Array [
      Object {
        astCacheSize: 0
        heapTotal: 26_079_232
        ownedBytes: 0
        ownedFileCount: 0
        pid: 1_636
        uptime: 0.1871088
      }
    ]
  }

Steps To Reproduce


  1. run yarn -s rome init
PS C:\Users\landi\code\typing-poc> yarn -s rome init

 project-rome/@internal/path/index.ts:196:9 internalError/request  INTERNAL  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
  × Expected relative file path but got: 

   9. ___R$project$rome$$internal$path$index_ts$AbsoluteFilePath.assertRelative (project-rome/@internal/path/index.ts:196:9)
  10. ___R$project$rome$$internal$path$index_ts$AbsoluteFilePath.toExplicitRelative (project-rome/@internal/path/index.ts:186:24)
  11. ___R$project$rome$$internal$consume$Consumer_ts$default.asRelativeFilePath (project-rome/@internal/consume/Consumer.ts:1095:15)
  12. ___R$project$rome$$internal$consume$Consumer_ts$default.asExplicitRelativeFilePath (project-rome/@internal/consume/Consumer.ts:1109:20)
  13. asExplicitRelativeFilePath (project-rome/@internal/codec-js-manifest/index.ts:512:29)
  14. ___R$$priv$project$rome$$internal$codec$js$manifest$index_ts$normalizeExports (project-rome/@internal/codec-js-manifest/index.ts:474:21)
  15. ___R$project$rome$$internal$codec$js$manifest$index_ts$normalizeManifest (project-rome/@internal/codec-js-manifest/index.ts:670:11)
  16. ___R$project$rome$$internal$core$server$fs$MemoryFileSystem_ts$default.declareManifestWithPossibleDiagnosticsThrow (project-rome/@internal/core/server/fs/MemoryFileSystem.ts:656:12)
  17. ___R$project$rome$$internal$diagnostics$wrap_ts$catchDiagnostics (project-rome/@internal/diagnostics/wrap.ts:29:16)
  18. ___R$project$rome$$internal$core$server$fs$MemoryFileSystem_ts$default.declareManifest (project-rome/@internal/core/server/fs/MemoryFileSystem.ts:627:24)
  19. ___R$project$rome$$internal$core$server$fs$MemoryFileSystem_ts$default.addFile (project-rome/@internal/core/server/fs/MemoryFileSystem.ts:926:3)
  20. declareItem (project-rome/@internal/core/server/fs/MemoryFileSystem.ts:760:4)
  21. ___R$project$rome$$internal$core$server$fs$MemoryFileSystem_ts$default.addDirectory (project-rome/@internal/core/server/fs/MemoryFileSystem.ts:771:5)
  22. declareItem (project-rome/@internal/core/server/fs/MemoryFileSystem.ts:762:4)
  23. Function.all
  24. ___R$project$rome$$internal$core$server$fs$MemoryFileSystem_ts$default.addDirectory (project-rome/@internal/core/server/fs/MemoryFileSystem.ts:777:2)
  25. declareItem (project-rome/@internal/core/server/fs/MemoryFileSystem.ts:762:4)
  26. Function.all
  27. ___R$project$rome$$internal$core$server$fs$MemoryFileSystem_ts$default.addDirectory (project-rome/@internal/core/server/fs/MemoryFileSystem.ts:777:2)
  28. ___R$project$rome$$internal$core$server$fs$MemoryFileSystem_ts$default.createWatcher (project-rome/@internal/core/server/fs/MemoryFileSystem.ts:297:3)
  and 8 others...

  ! This diagnostic was derived from an internal Rome error. Potential bug, please report if necessary.

━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
× Found 1 problem

Expected Results

Rome to successfully initiate a new project

unconfirmed
Source
picture kysley
👍10

All 9 comments

Got the same error

kirillgroshkov picture kirillgroshkov on 9 Aug 2020

I was able to reproduce the same error on a new project after adding https://github.com/uuidjs/uuid, i guess it has something to do with module resolution but i'm not really sure how to debug further.

n3xrodrigo picture n3xrodrigo on 9 Aug 2020

It's due to internal/path/index.ts where when constructing file paths we do the tilde exploding. When creating a relative path we need a way to signal to the create method that it's what we want and for it skip it, otherwise it looks like an absolute path of it starts with a tilde.

sebmck picture sebmck on 9 Aug 2020

I'm getting a similar error when trying to initialize rome in https://github.com/vega/ts-json-schema-generator. 

yarn rome init --apply
yarn run v1.22.4
$ /Users/dominik/Code/ts-json-schema-generator/node_modules/.bin/rome init --apply

 Welcome to Rome! Let's get you started...

[1/2] Generating lint config and apply formatting

 project-rome/@internal/path/index.ts:196:9 internalError/request  INTERNAL  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ✖ Expected relative file path but got:

  21. ___R$project$rome$$internal$path$index_ts$AbsoluteFilePath.assertRelative (project-rome/@internal/path/index.ts:196:9)
  22. ___R$project$rome$$internal$path$index_ts$AbsoluteFilePath.toExplicitRelative (project-rome/@internal/path/index.ts:186:24)
  23. ___R$project$rome$$internal$consume$Consumer_ts$default.asRelativeFilePath (project-rome/@internal/consume/Consumer.ts:1095:15)
  24. ___R$project$rome$$internal$consume$Consumer_ts$default.asExplicitRelativeFilePath (project-rome/@internal/consume/Consumer.ts:1109:20)
  25. asExplicitRelativeFilePath (project-rome/@internal/codec-js-manifest/index.ts:512:29)
  26. ___R$$priv$project$rome$$internal$codec$js$manifest$index_ts$normalizeExports (project-rome/@internal/codec-js-manifest/index.ts:474:21)
  27. ___R$project$rome$$internal$codec$js$manifest$index_ts$normalizeManifest (project-rome/@internal/codec-js-manifest/index.ts:670:11)
  28. ___R$project$rome$$internal$core$server$fs$MemoryFileSystem_ts$default.declareManifestWithPossibleDiagnosticsThrow (
      project-rome/@internal/core/server/fs/MemoryFileSystem.ts:656:12)
  29. ___R$project$rome$$internal$diagnostics$wrap_ts$catchDiagnostics (project-rome/@internal/diagnostics/wrap.ts:29:16)
  30. ___R$project$rome$$internal$core$server$fs$MemoryFileSystem_ts$default.declareManifest (project-rome/@internal/core/server/fs/MemoryFileSystem.ts:627:24)
  31. ___R$project$rome$$internal$core$server$fs$MemoryFileSystem_ts$default.addFile (project-rome/@internal/core/server/fs/MemoryFileSystem.ts:926:3)
  32. declareItem (project-rome/@internal/core/server/fs/MemoryFileSystem.ts:760:4)
  33. ___R$project$rome$$internal$core$server$fs$MemoryFileSystem_ts$default.addDirectory (project-rome/@internal/core/server/fs/MemoryFileSystem.ts:771:5)
  34. declareItem (project-rome/@internal/core/server/fs/MemoryFileSystem.ts:762:4)
  35. Function.all
  36. ___R$project$rome$$internal$core$server$fs$MemoryFileSystem_ts$default.addDirectory (project-rome/@internal/core/server/fs/MemoryFileSystem.ts:777:2)
  37. declareItem (project-rome/@internal/core/server/fs/MemoryFileSystem.ts:762:4)
  38. Function.all
  39. ___R$project$rome$$internal$core$server$fs$MemoryFileSystem_ts$default.addDirectory (project-rome/@internal/core/server/fs/MemoryFileSystem.ts:777:2)
  40. declareItem (project-rome/@internal/core/server/fs/MemoryFileSystem.ts:762:4)
  and 20 others...

  ⚠ This diagnostic was derived from an internal Rome error. Potential bug, please report if necessary.

━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

✖ Found 1 problem
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
domoritz picture domoritz on 9 Aug 2020

Reproducible at my end with the following steps:

  1. Create a new directory.
  2. From that directory, npm init -y
  3. npm install uuid
  4. npm install rome
  5. npx rome init

If you omit step 3, everything goes well.

igor-krupenja picture igor-krupenja on 9 Aug 2020
👍1

Same problem, adding Rome alongside empty Angular Project.

rome init --apply

 Welcome to Rome! Let's get you started... 

[1/2] Generating lint config and apply formatting

 project-rome/@internal/path/index.ts:721:8 internalError/request  INTERNAL  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ✖ Cannot construct a FilePath with zero segments

  13. ___R$$priv$project$rome$$internal$path$index_ts$parsePathSegments (project-rome/@internal/path/index.ts:721:8)
  14. ___R$project$rome$$internal$path$index_ts$RelativeFilePath.getUnique (project-rome/@internal/path/index.ts:400:21)
  15. ___R$project$rome$$internal$path$collections_ts$RelativeFilePathMap.set (project-rome/@internal/path/collections.ts:74:21)
  16. set (project-rome/@internal/codec-js-manifest/index.ts:475:10)
  17. ___R$project$rome$$internal$codec$js$manifest$index_ts$normalizeManifest (project-rome/@internal/codec-js-manifest/index.ts:670:11)
  18. ___R$project$rome$$internal$core$server$fs$MemoryFileSystem_ts$default.declareManifestWithPossibleDiagnosticsThrow (
      project-rome/@internal/core/server/fs/MemoryFileSystem.ts:656:12)
  19. ___R$project$rome$$internal$diagnostics$wrap_ts$catchDiagnostics (project-rome/@internal/diagnostics/wrap.ts:29:16)
  20. ___R$project$rome$$internal$core$server$fs$MemoryFileSystem_ts$default.declareManifest (project-rome/@internal/core/server/fs/MemoryFileSystem.ts:627:24)
  21. ___R$project$rome$$internal$core$server$fs$MemoryFileSystem_ts$default.addFile (project-rome/@internal/core/server/fs/MemoryFileSystem.ts:926:3)
  22. declareItem (project-rome/@internal/core/server/fs/MemoryFileSystem.ts:760:4)
  23. ___R$project$rome$$internal$core$server$fs$MemoryFileSystem_ts$default.addDirectory (project-rome/@internal/core/server/fs/MemoryFileSystem.ts:771:5)
  24. declareItem (project-rome/@internal/core/server/fs/MemoryFileSystem.ts:762:4)
  25. Function.all
  26. ___R$project$rome$$internal$core$server$fs$MemoryFileSystem_ts$default.addDirectory (project-rome/@internal/core/server/fs/MemoryFileSystem.ts:777:2)
  27. declareItem (project-rome/@internal/core/server/fs/MemoryFileSystem.ts:762:4)
  28. Function.all
  29. ___R$project$rome$$internal$core$server$fs$MemoryFileSystem_ts$default.addDirectory (project-rome/@internal/core/server/fs/MemoryFileSystem.ts:777:2)
  30. ___R$project$rome$$internal$core$server$fs$MemoryFileSystem_ts$default.createWatcher (project-rome/@internal/core/server/fs/MemoryFileSystem.ts:297:3)
  31. ___R$project$rome$$internal$core$server$fs$MemoryFileSystem_ts$default.watch (project-rome/@internal/core/server/fs/MemoryFileSystem.ts:537:23)
  32. ___R$project$rome$$internal$core$server$project$ProjectManager_ts$default.findProject (project-rome/@internal/core/server/project/ProjectManager.ts:891:5)
  and 12 others...

  ⚠ This diagnostic was derived from an internal Rome error. Potential bug, please report if necessary.

━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

✖ Found 1 problem
AntonioStipic picture AntonioStipic on 9 Aug 2020

FYI: if you install rome along with jest, you would encounter this error as while, as jest depends uuid package.

zhuangya picture zhuangya on 9 Aug 2020
👍1

It looks like Rome doesn't support nested conditions in the package.json exports field. For example uuid's exports field looks like this:

"exports": {
  ".": {
    "node": {
      "module": "./dist/esm-node/index.js",
      "require": "./dist/index.js",
      "import": "./wrapper.mjs"
    },
    "default": "./dist/esm-browser/index.js"
  },
  "./package.json": "./package.json"
}

Rome seems to be assuming here that after it gets to exports/./node, the value will be a path and not another object.

An even more mininal repro:

  1. npm init -y
  2. add "exports": { "node": { "module": "./index.js" } } to package.json
  3. npm install rome
  4. npx rome init
kyle1320 picture kyle1320 on 9 Aug 2020

1027 will fix this. Thanks for the repro steps everyone!

sebmck picture sebmck on 11 Aug 2020
Was this page helpful?
0 / 5 - 0 ratings

Related issues

sebmck picture sebmck  ·  3Comments
yamadayuki picture yamadayuki  ·  3Comments
josephspurrier picture josephspurrier  ·  4Comments
Kelbie picture Kelbie  ·  4Comments
RaviVaranasi picture RaviVaranasi  ·  5Comments