Skip to content

Add PDF coding standard document#4

Draft
Copilot wants to merge 2 commits intomainfrom
copilot/finish-task
Draft

Add PDF coding standard document#4
Copilot wants to merge 2 commits intomainfrom
copilot/finish-task

Conversation

Copy link

Copilot AI commented Feb 22, 2026

Addresses issue #2 — creates docs/pdf-coding-standard.md covering the project's conventions and standards.

Document covers:

  • Project architecture — layering rules (public API vs internal implementation)
  • Naming & class designPdf/Excel prefixes, sealed classes, internal constructors, readonly struct with IEquatable<T>
  • API patterns — fluent builder, static factory methods, default parameters
  • PDF 1.4 spec conventions — object tree structure, content stream operators (Tf, Td, Tj, rg), text positioning reset pattern, string escaping, InvariantCulture numeric formatting
  • Color handling — RGB 0–1 model, hex/ARGB parsing, clamping, fallback to black
  • Excel integration — zero-dependency xlsx parsing via ZipArchive/XDocument, style→font→color resolution chain, column scaling, pagination
  • Error handling — defensive defaults over exceptions, Math.Clamp, graceful fallback for malformed input
  • Testing{Method}_{Scenario}_{Expected} naming, in-memory xlsx generation, temp file cleanup
  • Security & dependencies — nullable enabled, zero external NuGet deps in library, allowed test deps
Original prompt

finish


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: shps951023 <12729184+shps951023@users.noreply.github.com>
Copilot AI changed the title [WIP] Finish incomplete task and finalize implementation Add PDF coding standard document Feb 22, 2026
Copilot AI requested a review from shps951023 February 22, 2026 11:29
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.

2 participants