npm package files: follow best practices, include only distributable source

[Copilot]

The files field was shipping everything—test files, dev configs, nonexistent LICENSE—instead of just the distributable source, types, and modules.

Changes

• Scoped source globs with explicit test exclusions:

  "files": [
    "src/**/*.ts",
    "src/**/*.tsx",
    "src/**/*.css",
    "src/**/*.d.ts",
    "!src/**/*.test.ts",
    "!src/**/*.test.tsx",
    "!src/test/",
    "public/",
    "index.html",
    "tsconfig.json",
    "tsconfig.app.json",
    "tsconfig.node.json",
    "vite.config.ts",
    "README.md"
  ]

• Removed dev-only configs from package: vitest.config.ts, eslint.config.js, typedoc.json, tsconfig.test.json
• Removed nonexistent LICENSE entry flagged in review
• Kept consumer-needed build configs: tsconfig solution/app/node + vite.config.ts (game template, not a library)

Package: 33 files (201 kB) → 17 files (102 kB).

[Copilot]

Pull request overview

This PR updates the project’s TypeScript/ESLint compilation targets to ES2025 and introduces TypeDoc-based API documentation generation, along with expanding the npm package “files” whitelist to include the intended source/config artifacts.

Changes:

• Bump TypeScript target/lib across tsconfig variants from ES2024 → ES2025 and update ESLint ecmaVersion to 2025.
• Add TypeDoc configuration (typedoc.json) and a docs:api script, and document it in the README.
• Add docs/api to .gitignore and expand package.json files whitelist.

Reviewed changes

Copilot reviewed 8 out of 11 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
typedoc.json	Adds TypeDoc configuration for generating API docs into docs/api with ISMS-related links/documents.
tsconfig.json	Updates base TS target/lib to ES2025.
tsconfig.app.json	Updates app TS target/lib to ES2025.
tsconfig.node.json	Updates node/build TS target/lib to ES2025.
tsconfig.test.json	Updates test TS target/lib to ES2025.
cypress/tsconfig.json	Updates Cypress TS target/lib to ES2025.
eslint.config.js	Updates ESLint parsing target to ecmaVersion: 2025.
README.md	Notes ES2025 targeting and documents npm run docs:api; adds TypeDoc to feature list.
package.json	Adds docs:api script, adds TypeDoc devDependency, and introduces an npm files whitelist.
package-lock.json	Locks TypeDoc and its transitive dependencies.
.gitignore	Ignores generated TypeDoc output directory docs/api.

[Copilot]

The npm files whitelist includes LICENSE, but this repository currently has no LICENSE (or LICENSE.md) file. That means npm pack/publish won’t include a license file as intended; either add the license file to the repo (and keep it in files) or remove/rename the entry to match the actual filename.

Suggested change

	"files": [
	"src/**/*.ts",
	"src/**/*.tsx",
	"src/**/*.css",
	"src/**/*.d.ts",
	"public/",
	"index.html",
	"tsconfig.json",
	"tsconfig.app.json",
	"tsconfig.node.json",
	"vite.config.ts",
	"README.md",
	"LICENSE"
	],
	"dependencies": {
	"files": [
	"src/**/*.ts",
	"src/**/*.tsx",
	"src/**/*.css",
	"src/**/*.d.ts",
	"public/",
	"index.html",
	"tsconfig.json",
	"tsconfig.app.json",
	"tsconfig.node.json",
	"vite.config.ts",
	"README.md"
	],
	"dependencies": {

[Copilot]

PR description says the npm package includes the “tsconfig files”, but the files whitelist only includes tsconfig.json/app/node (and not tsconfig.test.json or cypress/tsconfig.json). Either add the missing tsconfig files to the whitelist or adjust the description/intent so consumers aren’t surprised.

[pethers]

@copilot apply changes based on the comments in this thread analyse. wanted npm package to contain all types, code and modules following npm package best practices.

[Copilot]

@copilot apply changes based on the comments in this thread analyse. wanted npm package to contain all types, co...

Fixed in e56df58. The files field now uses src/ directory glob to include all types, code, tests, assets, and CSS. Added missing configs (tsconfig.test.json, vitest.config.ts, eslint.config.js, typedoc.json) and removed the nonexistent LICENSE entry. Verified with npm pack --dry-run — 33 files included covering all source modules and configuration.

[Copilot]

Pull request overview

Copilot reviewed 8 out of 11 changed files in this pull request and generated no new comments.

[pethers]

Updated in 4b80202. The files field now follows npm best practices:

Included (17 files, 102 kB):

• Source code: src/**/*.ts, src/**/*.tsx (with !src/**/*.test.ts, !src/**/*.test.tsx, !src/test/ exclusions)
• Type declarations: src/**/*.d.ts
• CSS modules: src/**/*.css
• Public assets: public/
• Entry: index.html
• Build configs: tsconfig.json, tsconfig.app.json, tsconfig.node.json, vite.config.ts
• Documentation: README.md

Excluded:

• Test files (*.test.ts, *.test.tsx, src/test/)
• Dev-only configs (vitest.config.ts, eslint.config.js, typedoc.json, tsconfig.test.json)
• Non-existent LICENSE