charlie: test

[shellscape]

@CharlieHelps are you here?

[charliecreates]

Yep — I’m here. What would you like help with?

[shellscape]

@CharlieHelps how about now?

[charliecreates]

Yep — I’m here and listening. What would you like me to do next?

[shellscape]

@CharlieHelps please review the open issues in this repo which report bugs and list any of those issues which don't have a corresponding pull request that's open. if it had a pull request, but that PR is closed, please note that. For each issue that is open which does not have an open PR, examine the validity of the issue against the codebase.

[charliecreates]

@shellscape

I did a pass over open issues (excluding PRs) and pulled out the ones that look like bug reports (based on title/body, not labels). Of the bug-report issues I found, only one currently has an open PR; the rest have no open PR at the moment.

Issue	Open PRs	Closed/Merged PRs	Quick validity triage (for issues w/ no open PR)
#1950 – TS non-watch builds leak TS watchers	—	—	likely-valid (plugin likely leaves a TS watch program alive even in non-watch; worth confirming with a minimal repro)
#1948 – TS sourcemap paths break with sourceRoot	—	—	likely-valid (doesn’t look like the plugin rewrites sourcemaps on master; likely TS/Rollup interaction)
#1946 – node-resolve “discards” a re-exported file	—	—	likely-valid (repro targets current node-resolve; no obvious fix/workaround in repo code)
#1873 – node-resolve external render path unexpected	—	—	unclear/outdated (seems expected that externals render the external id; output.paths / renderExternal may be the right knob)
#1860 – types/exports misconfigured for ESM	—	—	likely-valid (dual exports.import + single .d.ts can trip NodeNext/attw; may need conditional types or ESM-only packaging)
#1858 – TS plugin doesn’t drop .d.[ext].ts-style defs	#1949	—	(has an open PR)
#1851 – TypeScript conflict in Deno	—	—	needs-repro (no minimal repro in thread; likely Deno node-compat + conditional exports/types behavior)
#1672 – commonjs + core-js: userAgent.match is not a function	—	—	needs-repro (thread indicates repro isn’t minimal; should retest on latest @rollup/plugin-commonjs)
#986 – commonjs missing export / helpers issues	—	#1363	needs-repro (on master, helpers are served via virtual \\0commonjsHelpers.js; many failures end up being integration/publish pruning)

Notes / methodology

• Bug-report detection: skimmed open issue titles/bodies and marked items describing broken behavior (errors, expected vs actual, repro).
• PR linkage: preferred issue timeline cross-references/connected events; fell back to searching PRs for #<issue> mentions.
• Triage: quick spot-check of current master sources under relevant packages (typescript/node-resolve/commonjs). This isn’t a full repro run, just “does this still look plausible against current code?”

[shellscape]

@CharlieHelps why did you say that #986 needs a reproduction? Looks like it has one to me

[charliecreates]

Good catch — #986 does have a link, but it’s an old Replit repro from 2021 (Rollup v2 / plugin-commonjs v20) and it doesn’t clearly reproduce the current reports against recent versions. The later comments mostly describe symptoms (errors around "commonjsHelpers.js") without a runnable, minimal Rollup-only setup. That’s why I flagged it as still needing a repro: a small repo/script that fails on latest Rollup + @rollup/plugin-commonjs would make it actionable (ideally without Vite/WDS/WTR in the loop, unless that’s the only way it reproduces).