Move antlr4 to devDependencies in bnf package

[Copilot]

The antlr4 package was listed as a runtime dependency but is bundled into dist/index.mjs by rollup, unnecessarily requiring package consumers to install it.

Changes

• Moved antlr4 from dependencies to devDependencies in bnf/package.json
• Updated bnf/package-lock.json accordingly

The rollup configuration uses @rollup/plugin-node-resolve to bundle all antlr4 runtime code into the distribution file, making it self-contained. The package is only needed during build to generate parsers from the ANTLR grammar files and provide the runtime code that gets bundled.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• central.sonatype.com
  • Triggering command: /home/REDACTED/.local/bin/antlr4 antlr4 -Dlanguage=JavaScript Dsp.g4 -visitor -no-listener -o atsds_bnf (dns block)
  • Triggering command: /home/REDACTED/.local/bin/antlr4 antlr4 -Dlanguage=JavaScript Ds.g4 -visitor -no-listener -o atsds_bnf (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

Original prompt

---

This section details on the original issue you should resolve

<issue_title>bnf下的那个js包，antlr4被打包进dist了，不需要是依赖，只需要是开发依赖。</issue_title>
<issue_description></issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes bnf下的那个js包，antlr4被打包进dist了，不需要是依赖，只需要是开发依赖。 #102

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

[Copilot]

Pull request overview

This PR correctly moves the antlr4 package from runtime dependencies to devDependencies in the BNF package. The antlr4 runtime is bundled into the distribution file by rollup, so package consumers don't need to install it separately.

• Moved antlr4 from dependencies to devDependencies in bnf/package.json
• Updated bnf/package-lock.json to reflect the dependency category change

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File	Description
bnf/package.json	Moved antlr4 from dependencies to devDependencies since it's bundled by rollup
bnf/package-lock.json	Updated lock file to reflect antlr4 as a dev dependency (marked with "dev": true)

---

The changes are correct and well-justified. The rollup configuration uses @rollup/plugin-node-resolve to bundle all antlr4 runtime code into dist/index.mjs, making the distribution file self-contained. Since the package's exports point to this bundled file, consumers don't need antlr4 as a runtime dependency. The antlr4 package is only needed during development for:

1. Generating parsers from ANTLR grammar files (.g4 files)
2. Providing the runtime code that gets bundled into the distribution

No issues were found with this change.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.