Skip to content

fix(tag): include origin hints in illegal tag errors - #385

Merged
linkdata merged 2 commits into
mainfrom
fix/illegal-tag-caller-hint
Aug 24, 2026
Merged

fix(tag): include origin hints in illegal tag errors#385
linkdata merged 2 commits into
mainfrom
fix/illegal-tag-caller-hint

Conversation

@linkdata

@linkdata linkdata commented Aug 24, 2026

Copy link
Copy Markdown
Owner

Summary

  • add a bounded nearest-external-caller hint to illegal tag type errors
  • identify the innermost active TagGetter when its result contains an illegal type
  • skip JaWS, reflection, template, HTTP, runtime, and test dispatch frames
  • retain only type names, function names, file basenames, and line numbers
  • document that origin hints are best-effort while preserving errors.Is identity

Verification

  • go generate ./...
  • gofmt -l .
  • go vet ./...
  • staticcheck ./...
  • golangci-lint run
  • gosec ./...
  • go build ./...
  • JAWS_REQUIRE_NODE=1 go test -race ./...
  • JAWS_REQUIRE_NODE=1 go test ./...
  • go test -count=1 -gcflags=all=-l ./lib/tag
  • go test -count=1 -cover ./lib/tag (99.6%)
  • GOARCH=386 CGO_ENABLED=0 go test -exec=true ./... (cross-compiled on ARM64)

@linkdata linkdata changed the title fix(tag): include caller in illegal tag errors fix(tag): include origin hints in illegal tag errors Aug 24, 2026
@linkdata
linkdata merged commit b337341 into main Aug 24, 2026
7 checks passed
@linkdata
linkdata deleted the fix/illegal-tag-caller-hint branch August 24, 2026 10:55
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