Hacktoberfest 2026：维护者为十月标记出来的 issue，仍然开放、适合新手。 浏览 Hacktoberfest issue

fix: workspace-relative openInEditor paths resolve against cwd and double the package prefix in monorepos

已关闭
  • 哪里不对？
#1,097 2 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

维护者通常 1 天内回复

还没有人认领这个 Issue。

评估

难度
3/5
预计耗时
1-2 天
新手友好度
68/100
Issue 类型
缺陷
描述清晰度
基本清楚
活跃度
活跃
技术栈
nuxtjs, typescript
领域
developer-experience, tooling

调研方向

Start at the openInEditor entry point described in the issue and inspect how process.cwd(), nuxt.options.workspaceDir, and the existsSync check are used before openInEditorHooks. Reproduce with a pnpm workspace app under apps/web on Windows, then verify that the workspace-relative path opens the expected file and hooks are not blocked by an early failure.

由索引模型根据 Issue 内容生成。

描述

bug
🐛 The bug

In a pnpm monorepo, Component Inspector → Open in Editor fails with a doubled package path:

File not found: <repo>/apps/web/apps/web/app/features/.../SomeComponent.vue

Expected file:

<repo>/apps/web/app/features/.../SomeComponent.vue

(Same pattern for any package under apps/<name>/, e.g. previously apps/app → apps/app/apps/app/....)

Why

openInEditor resolves non-absolute paths with process.cwd() (the Nuxt package root, e.g. apps/web):

if (input.startsWith("./") || !ABSOLUTE_PATH_RE.test(input))
  input = resolve(process.cwd(), input)

The inspector often sends a workspace-relative path like:

apps/web/app/features/.../SomeComponent.vue

So:

cwd (= …/apps/web) + apps/web/app/...  →  …/apps/web/apps/web/app/...

existsSync then fails and returns before openInEditorHooks run, so hooks cannot fix the path.

Nuxt already exposes nuxt.options.workspaceDir for monorepos, but it is not used here.

🛠️ To reproduce

pnpm monorepo with Nuxt at apps/web → pnpm --filter dev (cwd=apps/web) → Component Inspector → Open in Editor → File not found with doubled apps/web/apps/web/...

🌈 Expected behavior

Resolve relative open-in-editor paths against nuxt.options.workspaceDir (and/or try cwd → rootDir → workspaceDir and pick the first path that exists), then open the file.

Ideally openInEditorHooks should also run (or path normalization should happen) before the hard existsSync failure, so consumers can fix monorepo paths.

ℹ️ Additional context
Detailed reproduction
  1. pnpm workspace with Nuxt app at apps/web (not repo root).
  2. Run pnpm --filter <app> dev so process.cwd() is apps/web.
  3. Enable Component Inspector, click a component, Open in Editor.
  4. Observe File not found: …/apps/web/apps/web/app/....

Minimal layout:

repo/
  pnpm-workspace.yaml   # packages: ['apps/*']
  apps/
    web/                # Nuxt app (cwd when running nuxt dev)
      app/
        components/
          Foo.vue
Environment
  • Nuxt: 4.5.x
  • @nuxt/devtools: 3.4.x (via Nuxt)
  • Package manager: pnpm workspace
  • OS: Windows 10/11
Workaround

Local Nuxt module wrapping the DevTools RPC openInEditor to rewrite the path (try cwd / rootDir / workspaceDir + undouble apps/<pkg>/apps/<pkg>/…) before calling the original handler.

Also set:

// nuxt.config.ts
workspaceDir: fileURLToPath(new URL('../..', import.meta.url)), // monorepo root
Related
主要语言
TypeScript
星标
3.3k
派生
217
平均合并
20 小时 18 分钟
30 天内合并 PR
17

环境准备

  • 没有 Dockerfile 或 Docker Compose 文件
  • 没有 Pull Request 模板
  • 阅读贡献指南

从这里开始

  1. 先读完整个 Issue，再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库，在一个分支上完成修改。
  4. 提交 Pull Request，并在描述里引用这个 Issue 编号。

nuxt/devtools 的其他 Issue

查看 nuxt/devtools 的全部 Issue

相似的 Issue

更多 TypeScript Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。