will this catch

'use server'
async function foo() { ... };
export const bar = foo;

?
FWIW i think next just does

if (typeof NAME === 'function') {
  registerServerReference(NAME, 'NAME' ...);
} else {
  errorBadExport('NAME');
}

for all exports because it's hard to analyze this statically

No, this does not cover all possible scenarios. It's pretty opinionated regarding what code style it supports, e.g. I also don't want to support default exports.

...or inline 'use server' directives

if (typeof NAME === 'function') {
  registerServerReference(NAME, 'NAME' ...);
} else {
  errorBadExport('NAME');
}

Hmm, I am considering this now...
Thanks for bringing this to my attention.

But why would you error in the else case?

i think it's because the expectation is that a "use server" module exports server references, and importing other things from it might act weird, because they won't get tranformed. like export const foo = 1 would work, but arbitrary non-function exports might not -- what if someone does export const dbClient = myDb.connect()? so easier to just blanket-ban than let the user run into puzzling errors i think

Yeah, makes sense.