Skip to content
Closed
Show file tree
Hide file tree
Changes from 10 commits
Commits
Show all changes
36 commits
Select commit Hold shift + click to select a range
c315580
add pnp resolution support
gun-yu Oct 11, 2025
99dc36c
remove fmt
gun-yu Oct 11, 2025
25b13ac
pnp internalize
gun-yu Oct 11, 2025
f24a170
remove useless code
gun-yu Oct 11, 2025
b4aaac8
remove useless code
gun-yu Oct 11, 2025
2373552
use go-json-experiment/json
gun-yu Oct 11, 2025
f09cda5
lint fix
gun-yu Oct 12, 2025
d00e36b
Merge branch 'main' into feat/add-pnp-resolver
gun-yu Oct 12, 2025
0a0f050
change test data path
gun-yu Oct 12, 2025
94a15b7
fix hereby format
gun-yu Oct 12, 2025
35d6188
fix hereby check:format
gun-yu Oct 13, 2025
35ee2ef
fix window test
gun-yu Oct 13, 2025
9463e11
Merge branch 'feat/add-pnp-resolver' of https://github.com/gun-yu/typ…
gun-yu Oct 13, 2025
dd08ae5
Merge branch 'main' into feat/add-pnp-resolver
gun-yu Oct 14, 2025
b368c1c
add License
gun-yu Oct 14, 2025
688455e
Merge branch 'feat/add-pnp-resolver' of https://github.com/gun-yu/typ…
gun-yu Oct 14, 2025
305fd49
remove filepath and utils
gun-yu Oct 16, 2025
f0ee55b
make regex more efficient
gun-yu Oct 16, 2025
ef6d713
fix lint
gun-yu Oct 16, 2025
8d165ca
Merge branch 'main' into feat/add-pnp-resolver
gun-yu Oct 16, 2025
ae272f1
apply codereview
gun-yu Oct 18, 2025
67752dc
generate error message
gun-yu Oct 18, 2025
1f06041
add error message
gun-yu Oct 18, 2025
a9dbc68
parse pnp.cjs file, regex to ast
gun-yu Oct 18, 2025
df40d62
make go version 1.25.0
gun-yu Oct 18, 2025
1485fe3
remove useless test
gun-yu Oct 18, 2025
bfa05c3
apply UnmarshalJSONFrom
gun-yu Oct 21, 2025
e697dce
apply UnmarshalJSONFrom in manifest
gun-yu Oct 21, 2025
1041038
os to vfs
gun-yu Oct 23, 2025
baf7d5a
Merge branch 'main' into feat/add-pnp-resolver
gun-yu Oct 24, 2025
01b7670
inject pnpResolutionConfig
gun-yu Oct 24, 2025
707270b
Merge branch 'feat/add-pnp-resolver' of https://github.com/gun-yu/typ…
gun-yu Oct 24, 2025
6b2f0c1
add comment for ata
gun-yu Oct 24, 2025
5c1cb56
remove useless test
gun-yu Oct 24, 2025
6747e23
change comment
gun-yu Oct 24, 2025
b9322a1
Merge branch 'main' into feat/add-pnp-resolver
gun-yu Oct 24, 2025
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion go.mod
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
module github.com/microsoft/typescript-go

go 1.25
go 1.25.1

require (
github.com/dlclark/regexp2 v1.11.5
Expand Down
2 changes: 1 addition & 1 deletion internal/compiler/fileloader.go
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,7 @@ func processAllProgramFiles(
includeProcessor: &includeProcessor{},
}
loader.addProjectReferenceTasks(singleThreaded)
loader.resolver = module.NewResolver(loader.projectReferenceFileMapper.host, compilerOptions, opts.TypingsLocation, opts.ProjectName)
loader.resolver = module.NewResolver(loader.projectReferenceFileMapper.host, compilerOptions, opts.TypingsLocation, opts.ProjectName, opts.Host.GetPNPResolutionConfig())
for index, rootFile := range rootFiles {
loader.addRootTask(rootFile, nil, &fileIncludeReason{kind: fileIncludeKindRootFile, data: index})
}
Expand Down
16 changes: 16 additions & 0 deletions internal/compiler/host.go
Original file line number Diff line number Diff line change
Expand Up @@ -3,11 +3,13 @@ package compiler
import (
"github.com/microsoft/typescript-go/internal/ast"
"github.com/microsoft/typescript-go/internal/core"
"github.com/microsoft/typescript-go/internal/module/pnp"
"github.com/microsoft/typescript-go/internal/parser"
"github.com/microsoft/typescript-go/internal/tsoptions"
"github.com/microsoft/typescript-go/internal/tspath"
"github.com/microsoft/typescript-go/internal/vfs"
"github.com/microsoft/typescript-go/internal/vfs/cachedvfs"
"github.com/microsoft/typescript-go/internal/vfs/pnpvfs"
)

type CompilerHost interface {
Expand All @@ -17,6 +19,7 @@ type CompilerHost interface {
Trace(msg string)
GetSourceFile(opts ast.SourceFileParseOptions) *ast.SourceFile
GetResolvedProjectReference(fileName string, path tspath.Path) *tsoptions.ParsedCommandLine
GetPNPResolutionConfig() *pnp.ResolutionConfig
}

var _ CompilerHost = (*compilerHost)(nil)
Expand All @@ -27,6 +30,7 @@ type compilerHost struct {
defaultLibraryPath string
extendedConfigCache tsoptions.ExtendedConfigCache
trace func(msg string)
pnpResolutionConfig *pnp.ResolutionConfig
}

func NewCachedFSCompilerHost(
Expand All @@ -49,12 +53,20 @@ func NewCompilerHost(
if trace == nil {
trace = func(msg string) {}
}

pnpResolutionConfig := TryGetPnpResolutionConfig(currentDirectory)

if pnpResolutionConfig != nil {
fs = pnpvfs.From(fs)
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On second thought, this way I wouldn’t be able to take advantage of the cached VFS, so I’m thinking of moving its location instead.

}

return &compilerHost{
currentDirectory: currentDirectory,
fs: fs,
defaultLibraryPath: defaultLibraryPath,
extendedConfigCache: extendedConfigCache,
trace: trace,
pnpResolutionConfig: pnpResolutionConfig,
}
}

Expand Down Expand Up @@ -86,3 +98,7 @@ func (h *compilerHost) GetResolvedProjectReference(fileName string, path tspath.
commandLine, _ := tsoptions.GetParsedCommandLineOfConfigFilePath(fileName, path, nil, h, h.extendedConfigCache)
return commandLine
}

func (h *compilerHost) GetPNPResolutionConfig() *pnp.ResolutionConfig {
return h.pnpResolutionConfig
}
54 changes: 54 additions & 0 deletions internal/compiler/pnp.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,54 @@
package compiler

import (
"io/fs"
"os"
"path/filepath"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The compiler should not depend on these; any accesses it makes must go through a provided FS.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(#1911 will enforce this in the future)

Copy link
Author

@gun-yu gun-yu Oct 23, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1041038
I replaced the implementation to use Yarn’s FindManifest API and switched from using the OS-based implementation to a VFS-based one.


module "github.com/microsoft/typescript-go/internal/module/pnp"
)

func TryGetPnpResolutionConfig(path string) *module.ResolutionConfig {
pnpManifestPath, err := findNearestPNPPath(path)
if err != nil {
return nil
}
pnpManifest, err := module.LoadPNPManifest(pnpManifestPath)
if err != nil {
return nil
}

return &module.ResolutionConfig{
Host: module.ResolutionHost{
FindPNPManifest: func(_ string) (*module.Manifest, error) {
return &pnpManifest, nil
},
},
}
}

func findNearestPNPPath(start string) (string, error) {
dir := start
if fi, err := os.Stat(start); err == nil {
if !fi.IsDir() {
dir = filepath.Dir(start)
}
} else {
dir = filepath.Dir(start)
}

for {
for _, name := range []string{".pnp.data.json", ".pnp.cjs", ".pnp.js"} {
candidate := filepath.Join(dir, name)
if fi, err := os.Stat(candidate); err == nil && !fi.IsDir() {
return candidate, nil
}
}
parent := filepath.Dir(dir)
if parent == dir {
break
}
dir = parent
}
return "", fs.ErrNotExist
}
5 changes: 5 additions & 0 deletions internal/execute/build/compilerHost.go
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ package build
import (
"github.com/microsoft/typescript-go/internal/ast"
"github.com/microsoft/typescript-go/internal/compiler"
"github.com/microsoft/typescript-go/internal/module/pnp"
"github.com/microsoft/typescript-go/internal/tsoptions"
"github.com/microsoft/typescript-go/internal/tspath"
"github.com/microsoft/typescript-go/internal/vfs"
Expand Down Expand Up @@ -38,3 +39,7 @@ func (h *compilerHost) GetSourceFile(opts ast.SourceFileParseOptions) *ast.Sourc
func (h *compilerHost) GetResolvedProjectReference(fileName string, path tspath.Path) *tsoptions.ParsedCommandLine {
return h.host.GetResolvedProjectReference(fileName, path)
}

func (h *compilerHost) GetPNPResolutionConfig() *pnp.ResolutionConfig {
return h.host.GetPNPResolutionConfig()
}
10 changes: 8 additions & 2 deletions internal/execute/build/host.go
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ import (
"github.com/microsoft/typescript-go/internal/compiler"
"github.com/microsoft/typescript-go/internal/execute/incremental"
"github.com/microsoft/typescript-go/internal/execute/tsc"
"github.com/microsoft/typescript-go/internal/module/pnp"
"github.com/microsoft/typescript-go/internal/tsoptions"
"github.com/microsoft/typescript-go/internal/tspath"
"github.com/microsoft/typescript-go/internal/vfs"
Expand All @@ -23,8 +24,9 @@ type host struct {
configTimes collections.SyncMap[tspath.Path, time.Duration]

// caches that stay as long as they are needed
resolvedReferences parseCache[tspath.Path, *tsoptions.ParsedCommandLine]
mTimes *collections.SyncMap[tspath.Path, time.Time]
resolvedReferences parseCache[tspath.Path, *tsoptions.ParsedCommandLine]
mTimes *collections.SyncMap[tspath.Path, time.Time]
pnpResolutionConfig *pnp.ResolutionConfig
}

var (
Expand Down Expand Up @@ -111,3 +113,7 @@ func (h *host) storeMTimeFromOldCache(file string, oldCache *collections.SyncMap
h.mTimes.Store(path, mTime)
}
}

func (h *host) GetPNPResolutionConfig() *pnp.ResolutionConfig {
return h.pnpResolutionConfig
}
62 changes: 62 additions & 0 deletions internal/module/pnp/builtins.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,62 @@
package pnp

import "slices"

var NodeJSBuiltins = []string{
"assert",
"assert/strict",
"async_hooks",
"buffer",
"child_process",
"cluster",
"console",
"constants",
"crypto",
"dgram",
"diagnostics_channel",
"dns",
"dns/promises",
"domain",
"events",
"fs",
"fs/promises",
"http",
"http2",
"https",
"inspector",
"module",
"net",
"os",
"path",
"path/posix",
"path/win32",
"perf_hooks",
"process",
"punycode",
"querystring",
"readline",
"readline/promises",
"repl",
"stream",
"stream/consumers",
"stream/promises",
"stream/web",
"string_decoder",
"sys",
"timers",
"timers/promises",
"tls",
"trace_events",
"tty",
"url",
"util",
"util/types",
"v8",
"vm",
"worker_threads",
"zlib",
}

func IsNodeJSBuiltin(s string) bool {
return slices.Contains(NodeJSBuiltins, s)
}
Loading
Loading