Module analyzer does not recognize triple-slash directive or @jsxImportSource pragma following hashbang comment

[0f-0b]

use deno_graph::{
    DefaultModuleAnalyzer, DefaultModuleParser, MediaType, ModuleAnalyzer as _, ModuleSpecifier,
};

fn main() {
    let specifier = ModuleSpecifier::parse("file:///a.tsx").unwrap();
    let source = &r#"
#!/usr/bin/env -S deno run
/// <reference path="./ref.d.ts" />
/* @jsxImportSource react */
"#[1..];
    let analyzer = DefaultModuleAnalyzer::new(&DefaultModuleParser);
    let module_info = analyzer
        .analyze(&specifier, source.into(), MediaType::Tsx)
        .unwrap();
    assert!(!module_info.ts_references.is_empty());
    assert!(module_info.jsx_import_source.is_some());
}

This program should not panic, but currently (with deno_graph 0.69.9) it panics unless the hashbang comment is removed.

[dsherret]

Is this something that doesn't work in Deno or doesn't work in deno_graph? I forget whose responsibility it is to remove/skip the hashbang.

[0f-0b]

I think Deno just supplies the source code to deno_graph unmodified, and it's up to deno_graph to skip the hashbang. The linked PR implements this.