Skip to content

Annotations ARE implemented here, and three absences are somewhere else - #35

Merged
tannevaled merged 1 commit into
mainfrom
annotations-are-implemented-here
Sep 26, 2026
Merged

tannevaled merged 1 commit into
mainfrom
annotations-are-implemented-here

Conversation

@tannevaled

Copy link
Copy Markdown
Contributor

Encryption, tagged/PDF-A, forms and annotations are not yet implemented.

One line denying four things. Measured in this repository and across the
organisation:

annotations page.go:83 AddLink, :90 AddNamedDest, :97 AddNamedLink write a clickable /Link. The sentence is simply false.
encryption absent here (0 non-test files), present in go-pdfkit/reader: RC4-40 through AES-256 read, AES-128 and AES-256 written
forms absent here (no AcroForm), present in go-pdfkit/forms, and xfa reads the other kind
tagged / PDF-A PDF/A absent everywhere; a structure tree is preserved by go-pdfkit/ops and created by nothing

"Not implemented" and "not in this package" are different sentences, and the
reader who needs the second is sent away by the first. Each bullet now says which
one it means, and where the answer is.

Companion pull requests: go-pdfkit/.github#19 (the same claim on the profile)
and go-pdfkit/go-pdfkit.github.io#12 (the five modules the site never named).

🤖 Generated with Claude Code

"Encryption, tagged/PDF-A, forms and annotations are not yet implemented" is
one line denying four things. Measured in this repository and the organisation:

  annotations   page.go:83 AddLink, :90 AddNamedDest, :97 AddNamedLink write a
                clickable /Link. The sentence is simply false.
  encryption    absent here (0 non-test files), present in go-pdfkit/reader:
                RC4-40 through AES-256 read, AES-128 and AES-256 written.
  forms         absent here (no AcroForm), present in go-pdfkit/forms and xfa.
  tagged/PDF-A  PDF/A absent everywhere; a structure tree is preserved by
                go-pdfkit/ops and created by nothing.

"Not implemented" and "not in this package" are different sentences, and the
reader who needs the second is sent away by the first. Each bullet now says
which one it means and where the answer is.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@tannevaled
tannevaled merged commit e7afcf7 into main Sep 26, 2026
1 check passed
@tannevaled
tannevaled deleted the annotations-are-implemented-here branch September 26, 2026 15:39
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