Skip to content

refactor(tools): deduplicate export transformers #2358

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
fengmk2 merged 5 commits into from
Aug 6, 2026
Merged

refactor(tools): deduplicate export transformers #2358

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
ce46c39
refactor(tools): deduplicate export transformers
jong-kyung Aug 6, 2026
5c5f3b2
Merge branch 'main' into refactor/dedupe-export-transform
jong-kyung Aug 6, 2026
4b4a11b
Merge branch 'main' into refactor/dedupe-export-transform
jong-kyung Aug 6, 2026
a3ccdbd
Merge branch 'main' into refactor/dedupe-export-transform
jong-kyung Aug 6, 2026
bd9c93d
Merge branch 'main' into refactor/dedupe-export-transform
jong-kyung Aug 6, 2026
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
192 changes: 61 additions & 131 deletions packages/tools/src/sync-remote-deps.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -144,156 +144,86 @@ function cloneRepo(repoUrl: string, dir: string, branch: string, hash?: string)
}
}

function transformRolldownExport(exportPath: string, exportValue: unknown): [string, ExportValue] {
// Skip package.json
if (exportPath === './package.json') {
return ['', null];
}

// Transform export path: . -> ./rolldown, ./foo -> ./rolldown/foo
const newExportPath = exportPath === '.' ? './rolldown' : `./rolldown${exportPath.slice(1)}`;

// Transform export value
const transformValue = (value: unknown): ExportValue => {
if (typeof value === 'string') {
// Skip 'dev' condition paths that point to src
if (value.startsWith('./src/')) {
return null;
}
// Transform dist paths
return value.replace(/^\.\/dist\//, './dist/rolldown/');
function createExportTransformer(exportPathPrefix: string, distPathPrefix: string) {
return (exportPath: string, exportValue: unknown): [string, ExportValue] => {
if (exportPath === './package.json') {
return ['', null];
}

if (value && typeof value === 'object') {
const result: Record<string, unknown> = {};
for (const [key, val] of Object.entries(value)) {
// Skip 'dev' condition
if (key === 'dev') {
continue;
}
const newExportPath =
exportPath === '.' ? exportPathPrefix : `${exportPathPrefix}${exportPath.slice(1)}`;

const transformed = transformValue(val);
if (transformed !== null) {
result[key] = transformed;
const transformValue = (value: unknown): ExportValue => {
if (typeof value === 'string') {
if (value.startsWith('./src/')) {
return null;
}
return value.replace(/^\.\/dist\//, distPathPrefix);
}
return Object.keys(result).length > 0 ? (result as ExportValue) : null;
}

return value as ExportValue;
};

const newValue = transformValue(exportValue);

// Handle string values or add types if missing
if (typeof newValue === 'string') {
// Convert string to object with default and types
if (newValue.endsWith('.mjs')) {
return [
newExportPath,
{
default: newValue,
types: newValue.replace(/\.mjs$/, '.d.mts'),
},
];
} else if (newValue.endsWith('.js')) {
return [
newExportPath,
{
default: newValue,
types: newValue.replace(/\.js$/, '.d.ts'),
},
];
}
return [newExportPath, newValue];
}

if (newValue && typeof newValue === 'object') {
const importPath = ('import' in newValue ? newValue.import : newValue.default) as
| string
| undefined;
if (importPath && !('types' in newValue)) {
if (importPath.endsWith('.mjs')) {
newValue.types = importPath.replace(/\.mjs$/, '.d.mts');
} else if (importPath.endsWith('.js')) {
newValue.types = importPath.replace(/\.js$/, '.d.ts');
if (value && typeof value === 'object') {
const result: Record<string, unknown> = {};
for (const [key, val] of Object.entries(value)) {
if (key === 'dev') {
continue;
}
const transformed = transformValue(val);
if (transformed !== null) {
result[key] = transformed;
}
}
return Object.keys(result).length > 0 ? (result as ExportValue) : null;
}
}
}

return [newExportPath, newValue];
}

function transformPluginutilsExport(
exportPath: string,
exportValue: unknown,
): [string, ExportValue] {
// Skip package.json
if (exportPath === './package.json') {
return ['', null];
}

// Transform . -> ./rolldown/pluginutils
const newExportPath =
exportPath === '.' ? './rolldown/pluginutils' : `./rolldown/pluginutils${exportPath.slice(1)}`;
return value as ExportValue;
};

// Transform paths
const transformValue = (value: unknown): ExportValue => {
if (typeof value === 'string') {
if (value.startsWith('./src/')) {
return null;
const newValue = transformValue(exportValue);

if (typeof newValue === 'string') {
if (newValue.endsWith('.mjs')) {
return [
newExportPath,
{
default: newValue,
types: newValue.replace(/\.mjs$/, '.d.mts'),
},
];
} else if (newValue.endsWith('.js')) {
return [
newExportPath,
{
default: newValue,
types: newValue.replace(/\.js$/, '.d.ts'),
},
];
}
return value.replace(/^\.\/dist\//, './dist/pluginutils/');
return [newExportPath, newValue];
}

if (value && typeof value === 'object') {
const result: Record<string, unknown> = {};
for (const [key, val] of Object.entries(value)) {
if (key === 'dev') {
continue;
}
const transformed = transformValue(val);
if (transformed !== null) {
result[key] = transformed;
if (newValue && typeof newValue === 'object') {
const importPath = ('import' in newValue ? newValue.import : newValue.default) as
| string
| undefined;
if (importPath && !('types' in newValue)) {
if (importPath.endsWith('.mjs')) {
newValue.types = importPath.replace(/\.mjs$/, '.d.mts');
} else if (importPath.endsWith('.js')) {
newValue.types = importPath.replace(/\.js$/, '.d.ts');
}
}
return Object.keys(result).length > 0 ? (result as ExportValue) : null;
}

return value as ExportValue;
};

const newValue = transformValue(exportValue);

// Handle string values or add types if missing
if (typeof newValue === 'string') {
// Convert string to object with default and types
if (newValue.endsWith('.js')) {
return [
newExportPath,
{
default: newValue,
types: newValue.replace(/\.js$/, '.d.ts'),
},
];
}
return [newExportPath, newValue];
}

if (newValue && typeof newValue === 'object') {
const importPath = ('import' in newValue ? newValue.import : newValue.default) as
| string
| undefined;
if (importPath && !('types' in newValue)) {
if (importPath.endsWith('.js')) {
newValue.types = importPath.replace(/\.js$/, '.d.ts');
}
}
}

return [newExportPath, newValue];
};
}

const transformRolldownExport = createExportTransformer('./rolldown', './dist/rolldown/');
const transformPluginutilsExport = createExportTransformer(
'./rolldown/pluginutils',
'./dist/pluginutils/',
);

function transformViteExport(exportPath: string, exportValue: unknown): [string, ExportValue] {
// Skip package.json
if (exportPath === './package.json') {
Expand Down
Loading