Skip to content

chore: add SPDX license headers and NOTICE file - #30

Merged
amrtgaber merged 1 commit into
mainfrom
chore/license-headers
Jun 15, 2026
Merged

chore: add SPDX license headers and NOTICE file#30
amrtgaber merged 1 commit into
mainfrom
chore/license-headers

Conversation

@amrtgaber

Copy link
Copy Markdown
Contributor

What

  • Add SPDX license headers (Apache-2.0, AG Technology Group LLC) to every tracked TypeScript/JS source file (30 files).
  • Add an attribution-only NOTICE at the repo root.

Enforcement (CI-only)

  • scripts/license-header.mjs — dependency-free Node check/stamp tool.
    • The CI lint-and-test job runs pnpm run license:check.
    • Run pnpm run license:fix to stamp new files locally.
  • Deliberately not wired into husky/lint-staged. This is a template repo: a local hook would be inherited by every clone and have to be ripped out. The check only verifies that an SPDX-License-Identifier line exists, so a downstream fork with its own copyright still passes.

Skipped

src/routeTree.gen.ts and src/api/generated/ (generated), plus non-source files.

Verification

prettier --check, eslint (0 errors), tsc --noEmit, and vitest (2) all passing; build succeeds; the check is idempotent (30 stamped / 1 skipped).

Stamp every TypeScript/JS source file with SPDX headers (Apache-2.0,
AG Technology Group LLC) and add an attribution-only NOTICE. Generated
code (src/routeTree.gen.ts, src/api/generated) is skipped.

Enforcement is CI-only (no local hook): the lint-and-test job runs
`pnpm run license:check`, which verifies each tracked source file
carries an SPDX identifier (run `pnpm run license:fix` to stamp new
files). Kept out of husky/lint-staged so anyone cloning the template
doesn't inherit a local hook they'd have to strip out.
@amrtgaber
amrtgaber merged commit f07dc13 into main Jun 15, 2026
2 checks passed
@amrtgaber
amrtgaber deleted the chore/license-headers branch June 15, 2026 02:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant