Skip to content

feat: relocate vite concerns from esbuild-plugin to vite-plugin - #4633

Merged
GordonSmith merged 1 commit into
hpcc-systems:mainfrom
GordonSmith:VITE_REFACTOR
Aug 19, 2026
Merged

feat: relocate vite concerns from esbuild-plugin to vite-plugin#4633
GordonSmith merged 1 commit into
hpcc-systems:mainfrom
GordonSmith:VITE_REFACTOR

Conversation

@GordonSmith

Copy link
Copy Markdown
Member

Checklist:

  • The commit message is properly formatted and free of typos.
    • The commit message title makes sense in a changelog, by itself.
    • The commit message includes a "fixes" reference if appropriate.
    • The commit is signed.
  • The change has been fully tested:
    • I have viewed all related gallery items
    • I have viewed all related dermatology items
  • There are no similar instances of the same problem that should be addressed
    • I have addressed them here
    • I have raised new issues to address them separately

Testing:

@GordonSmith
GordonSmith force-pushed the VITE_REFACTOR branch 2 times, most recently from 897db3e to 6387ff1 Compare August 19, 2026 12:07
});
const expr = build(payload.expr);
lines.push(
`export const __dojoEvaluate = () => ${expr};`,
payload.deps.forEach((dep, i) => {
const absMid = dep.split("!").map(part => resolver.toAbsMid(part, payload.reference)).join("!");
lines.push(`import __d${i} from ${JSON.stringify(dep)};`);
entries.push(`${JSON.stringify(absMid)}: __dojoDep(() => __d${i}, ${JSON.stringify(absMid)})`);
const entries: string[] = [];
deps.forEach((dep, i) => {
lines.push(`import __d${i} from ${JSON.stringify(dep)};`);
entries.push(`${JSON.stringify(dep)}: __dojoDep(() => __d${i}, ${JSON.stringify(dep)})`);
const requested = resolver.toAbsMid(mid, reference);
noteAlias(canonicalOf(resolved.id), requested);
const unwrapDefault = needsInterop && options.esmInterop !== "namespace";
return `__dojoRt.__dojoDep(() => ${name}, ${JSON.stringify(requested)}${unwrapDefault ? ", 1" : ""})`;
noteAlias(canonicalOf(resolved.id), requested);
const isRuntimeHas = resolved.id.startsWith(PREFIX + "has:");
target.push(
`${JSON.stringify(mid)}: {load: () => import(${JSON.stringify(resolved.id)}), absMid: ${JSON.stringify(requested)}${isRuntimeHas ? ", runtimeHas: true" : ""}}`
noteAlias(canonicalOf(resolved.id), requested);
const isRuntimeHas = resolved.id.startsWith(PREFIX + "has:");
target.push(
`${JSON.stringify(mid)}: {load: () => import(${JSON.stringify(resolved.id)}), absMid: ${JSON.stringify(requested)}${isRuntimeHas ? ", runtimeHas: true" : ""}}`
@GordonSmith
GordonSmith force-pushed the VITE_REFACTOR branch 2 times, most recently from ef59b94 to 226483c Compare August 19, 2026 12:20
Signed-off-by: Gordon Smith <GordonJSmith@gmail.com>
@GordonSmith
GordonSmith merged commit dc4b404 into hpcc-systems:main Aug 19, 2026
5 checks passed
@GordonSmith
GordonSmith deleted the VITE_REFACTOR branch August 19, 2026 12:51
@github-actions github-actions Bot mentioned this pull request Aug 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants