Skip to content

fix: report Notification BootId as stored; remove unsourced doc claims - #47

Merged
abrignoni merged 1 commit into
mainfrom
fix/doc-claims-audit
Jul 31, 2026
Merged

fix: report Notification BootId as stored; remove unsourced doc claims#47
abrignoni merged 1 commit into
mainfrom
fix/doc-claims-audit

Conversation

@abrignoni

Copy link
Copy Markdown
Owner

Fixes from a documentation audit of artifacts touched 2026-07-29/30, applying the standard that notes and descriptions must not state what data means unless the data (or a cited source) proves it.

Behavior change

windowsSystem / Notifications: Notification.BootId was converted as a Windows FILETIME and reported as "Boot ID Time (UTC)". Nothing in the repo establishes the field is a timestamp — the lab validation never covered BootId, the unit test only proved the conversion of a synthetic FILETIME it inserted itself, and public wpndatabase research describes BootId as a boot sequence identifier. A real counter value would have rendered as a bogus 1601-era timestamp in front of an examiner. The column is now "Boot ID (database value)", reported as stored, with a test asserting the raw pass-through.

Documentation-only fixes

  • Notifications note: the FILETIME/UTC claim now covers only arrival and expiry (lab validated); Boot ID is stated as reported-as-stored.
  • SetupAPI note: "device-local because the log records no UTC offset" was a fact stated with a justification that doesn't support it (a missing offset shows ambiguity, not local time); now attributed to Microsoft's documented SetupAPI logging behavior.
  • Photos note: "Alternate Date Taken is an app-maintained fallback" was an unsourced purpose attribution; now described as a separate app-recorded date whose derivation is not documented.
  • Alarms: "Days of Week (bitmask)" asserted an encoding never established; now "(database value)", and the notes state that Recurring is derived from a nonzero Days of Week only when the source carries no IsRecurring value.

All 10 Windows artifact tests pass; lint gate clean.

🤖 Generated with Claude Code

Documentation audit fixes. One behavior change: BootId was being
converted as a Windows FILETIME and reported as 'Boot ID Time (UTC)',
but nothing establishes the field is a timestamp (public wpndatabase
research describes it as a boot sequence identifier), so a counter
value would have rendered as a bogus 1601-era time. It is now reported
as stored, with a test asserting the raw pass-through.

Doc-only fixes:
- windowsSystem notifications note no longer includes boot values in
  the FILETIME claim; arrival/expiry keep it (lab validated).
- setupapi note attributed device-local times to Microsoft's documented
  SetupAPI logging behavior instead of inferring it from the missing
  UTC offset.
- windowsApps Photos note: Alternate Date Taken described as a separate
  app-recorded date whose derivation is not documented, not a
  'fallback'.
- windowsApps Alarms: 'Days of Week (bitmask)' header now '(database
  value)' since the encoding is not established, and the notes state
  when Recurring is derived rather than stored.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@abrignoni
abrignoni merged commit 4861622 into main Jul 31, 2026
5 checks passed
@abrignoni
abrignoni deleted the fix/doc-claims-audit branch July 31, 2026 16:19
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