Move the license-header tool out of the Go Server module (#1018)

Author: longquanzheng

.dockerignore

@@ -41,3 +41,4 @@ sdk-go/coverage/
 server/coverage/
 server/coverage-raw/
 server/all-in-one/bin/
+script/licenseheaders/copyright/copyright

.github/workflows/copyright-ci.yml

@@ -66,7 +66,9 @@ jobs:
           fetch-depth: 0
       - uses: actions/setup-go@v7
         with:
-          go-version-file: server/go.mod
+          go-version-file: script/licenseheaders/copyright/go.mod
           cache: false
+      - name: Test license header tool
+        run: make copyright-test
       - name: Verify license headers
         run: make copyright-check

.gitignore

@@ -68,3 +68,4 @@ sdk-typescript/coverage/
 *.log
 test.log
 server/all-in-one/bin/
+script/licenseheaders/copyright/copyright

CONTRIBUTING.md

@@ -149,8 +149,12 @@ make copyright-check   # verify classifications, body hashes, and headers
 
 Skip generated trees (`**/gen/**`, `*.pb.go`, `*_pb.go`, `*.gen.*`). Prefer
 `make copyright` over hand-copying when adding files. It upgrades a modified
-`legacy-only` file to `mixed` without replacing its legacy notice. CI runs
-`make copyright-check` via [`.github/workflows/copyright-ci.yml`](.github/workflows/copyright-ci.yml).
+`legacy-only` file to `mixed` without replacing its legacy notice.
+
+The standalone Go tool lives in
+[`script/licenseheaders/copyright/`](script/licenseheaders/copyright/).
+Run `make copyright-test` to test its classification and rendering behavior.
+CI runs `make copyright-check` via [`.github/workflows/copyright-ci.yml`](.github/workflows/copyright-ci.yml).
 
 Files absent from the cutoff manifest use the new header. File-level renames
 and copies with at least 20% similarity retain source lineage; non-new copies

Makefile

@@ -8,7 +8,7 @@ GENERATED_CODE_PATHS := \
 	sdk-python/dex/dexpb \
 	sdk-typescript/src/gen
 
-.PHONY: help ci-runner-check copyright copyright-check generated-code generated-code-check githooks docs-prose-check docs-prose-fix release-tooling-test
+.PHONY: help ci-runner-check copyright copyright-check copyright-test generated-code generated-code-check githooks docs-prose-check docs-prose-fix release-tooling-test
 
 release-tooling-test: ## Verify changed-component selection and release preflight
 	PYTHONDONTWRITEBYTECODE=1 python3 -m unittest discover -s script/release -p '*_test.py'
@@ -33,10 +33,13 @@ ci-runner-check: ## Verify CI workflows use GitHub-hosted runners
 	bash .github/scripts/check-ci-runners.sh
 
 copyright: ## Add or upgrade license headers using the legacy manifest
-	cd server && GOWORK=off go run ./cmd/tools/copyright -rootDir "$(ROOT_DIR)"
+	cd script/licenseheaders/copyright && GOWORK=off go run . -rootDir "$(ROOT_DIR)"
 
 copyright-check: ## Verify license classifications, body hashes, and headers
-	cd server && GOWORK=off go run ./cmd/tools/copyright -rootDir "$(ROOT_DIR)" -verifyOnly
+	cd script/licenseheaders/copyright && GOWORK=off go run . -rootDir "$(ROOT_DIR)" -verifyOnly
+
+copyright-test: ## Test license header classification and rendering
+	cd script/licenseheaders/copyright && GOWORK=off go test ./...
 
 generated-code: ## Regenerate all checked-in code derived from protos
 	$(MAKE) -C protos proto

script/licenseheaders/copyright/README.md

@@ -0,0 +1,26 @@
+# License header tool
+
+Adds and verifies repository license headers using the policy, templates, and
+legacy manifest in the parent directory. This standalone Go module does not
+depend on the Dex Server.
+
+From the repository root:
+
+```bash
+make copyright
+make copyright-check
+make copyright-test
+```
+
+To process selected files, pass repository-relative paths from this directory:
+
+```bash
+GOWORK=off go run . -rootDir ../../.. -filePaths path/to/file.go
+GOWORK=off go run . -rootDir ../../.. -verifyOnly
+```
+
+The implementation and license classifications are preserved from the original
+Server tool. Git history supplies rename and copy lineage from the legacy cutoff,
+so CI must fetch the full repository history. The tool's own `main.go` keeps its
+`mixed` classification through `reviewed_mixed_paths` in `../policy.json`,
+because the repository-wide diff exceeds Git's default rename limit.

script/licenseheaders/copyright/go.mod

@@ -0,0 +1,11 @@
+module github.com/superdurable/dex/script/licenseheaders/copyright
+
+go 1.26.0
+
+require github.com/stretchr/testify v1.11.1
+
+require (
+	github.com/davecgh/go-spew v1.1.1 // indirect
+	github.com/pmezard/go-difflib v1.0.0 // indirect
+	gopkg.in/yaml.v3 v3.0.1 // indirect
+)

script/licenseheaders/copyright/go.sum

@@ -0,0 +1,10 @@
+github.com/davecgh/go-spew v1.1.1 h1:vj9j/u1bqnvCEfJOwUhtlOARqs3+rkHYY13jYWTU97c=
+github.com/davecgh/go-spew v1.1.1/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=
+github.com/pmezard/go-difflib v1.0.0 h1:4DBwDE0NGyQoBHbLQYPwSUPoCMWR5BEzIk/f1lZbAQM=
+github.com/pmezard/go-difflib v1.0.0/go.mod h1:iKH77koFhYxTK1pcRnkKkqfTogsbg7gZNVY4sRDYZ/4=
+github.com/stretchr/testify v1.11.1 h1:7s2iGBzp5EwR7/aIZr8ao5+dra3wiQyKjjFuvgVKu7U=
+github.com/stretchr/testify v1.11.1/go.mod h1:wZwfW3scLgRK+23gO65QZefKpKQRnfz6sD981Nm4B6U=
+gopkg.in/check.v1 v0.0.0-20161208181325-20d25e280405 h1:yhCVgyC4o1eVCa2tZl7eS0r+SDo693bJlVdllGtEeKM=
+gopkg.in/check.v1 v0.0.0-20161208181325-20d25e280405/go.mod h1:Co6ibVJAznAaIkqp8huTwlJQCZ016jof/cbN4VW5Yz0=
+gopkg.in/yaml.v3 v3.0.1 h1:fxVm/GzAzEWqLHuvctI91KS9hhNmmWOoWu0XTYJS7CA=
+gopkg.in/yaml.v3 v3.0.1/go.mod h1:K4uyk7z7BCEPqu6E+C64Yfv1cQ7kz7rIZviUmN+EgEM=

server/cmd/tools/copyright/main.go script/licenseheaders/copyright/main.goserver/cmd/tools/copyright/main.go renamed to script/licenseheaders/copyright/main.go

@@ -108,7 +108,7 @@ func TestHeaderTaskUpgradesLegacyAndCreatesNewHeader(t *testing.T) {
 func copyTestTemplates(t *testing.T, repositoryRoot string) {
 	_, currentFile, _, ok := runtime.Caller(0)
 	require.True(t, ok)
-	sourceDir := filepath.Join(filepath.Dir(currentFile), "..", "..", "..", "..", "script", "licenseheaders")
+	sourceDir := filepath.Join(filepath.Dir(currentFile), "..")
 	destinationDir := filepath.Join(repositoryRoot, "script", "licenseheaders")
 	require.NoError(t, os.MkdirAll(destinationDir, 0o755))
 	for _, fileName := range []string{