Audit: module docs and stale comments (comments are docs) #863
Open
opened 2026-10-02 16:47:34 +00:00 by kayg
·
11 comments
No Branch/Tag specified
dev
wip/mailghost2-1094
wip/mailghost-1094
wip/kbpreview2-1118
wip/kbpreview-1118
wip/kanban-1092
wip/importhang-1121
wip/hiderev-1153
wip/hide4-1153
wip/hide3-1153
wip/hide2-1153
wip/hide-1153
wip/editreg-1132
wip/editorrail3-1113
wip/editorrail2-1113
wip/editorrail-1113
wip/e2e-b2-1071
wip/e2e-b-1071
wip/draw4-1101
wip/draw3-1101
wip/draw2-1101
wip/draw-1101
wip/directory-1199
wip/delete-1119
wip/collabloss-1197
wip/cards2-1083
wip/cards-1083
wip/canvas-visual
wip/canvasvis2-976
wip/calhdr-1112
wip/calcards-1115
wip/browserfix
wip/blocks-1125
wip/allday-1107
wip/agenda-decks
wip/agenda-1086
wip/adv7c-1105
wip/txentry-1198
wip/trayicons2-1095
wip/trayicons-1095
wip/tagperf-1186
wip/sidebar3-1094
wip/segmented-1200
wip/rev2-webperf
wip/rev2-money-ident
wip/previewcard-1098
job/collabloss-1197
wip/palette2-1123
wip/palette-1093
wip/onboard2-1141
job/restyle-settings
wip/onboard-1141.aborted-early
wip/onboard-1141
wip/notifloop-1194
wip/nlpchip-1127
wip/morph-1104
wip/merge-round-7c5
wip/merge-round-7c4
wip/merge-round-7c3
wip/merge-round-7c2
wip/merge-round-7c
wip/mchrome-1084
job/restyle-files
job/tagdnd-1187
job/merge30
job/perf-1124
job/tocrail-1191
job/cards-1179
wip/cards2-1179
wip/cards-1179
job/segmented-1200
wip/tocrail-1191
job/hide-1153
wip/tagdnd-1187
wip/restyle-files
wip/perf-1124
wip/merge30j
job/onboard-1141
job/restyle-notes
wip/restyle-notes
job/wizchoices-1140
job/adv-1202
job/notifloop-1194
wip/wizchoices-1140
wip/restyle-1190
job/moneyfmt-1180
job/txentry-1198
wip/moneyfmt2-1180
wip/moneyfmt-1180-r
wip/moneyfmt-1180
job/tagperf-1186
job/pillglass-1189
job/flags-1181
wip/flags-1181
job/restyle-1190
job/restyle-mailmoney
job/restyle-search
job/settingsreg-1195
job/wizard-1140
site/website
wip/wizardrev2-1140
wip/wizardrev-1140
wip/wizard5-1140
wip/wizard4-1140
wip/wizard3-1140
wip/wizard2-1140
wip/wizard-1140
wip/pillglass-1189
wip/settingsreg-1195
job/merge29
job/fu-1171
wip/merge29j
wip/fu-1171
job/fu-1166
job/directory-1199
job/txresearch-1188
wip/fu-1166
job/merge28
job/search-1066
wip/search-1066
wip/merge28j
job/gateslot-1182
job/bulkimport-1157
job/mailnet-1160
wip/mailnetrev-1160
wip/mailnet-1160
wip/bulkrev-1157
wip/bulkimport-1157
job/startup-1161
wip/startup-1161
job/merge27
job/linkcards-1151
wip/linkcards3-1151
wip/linkcards2-1151
wip/linkcards-1151
job/traydate-1144
wip/traydate3-1144
wip/traydate2-1144
wip/traydate-1144
job/draw-1101
wip/merge27j
job/blockpill-1152
wip/blockpill3-1152
wip/blockpill2-1152
wip/blockpill-1152
job/minihover-1149
wip/minihover2-1149
wip/minihover-1149
job/merge25
wip/merge25-r
wip/merge25b
wip/merge25
job/inspector-1129
job/tags-1110
wip/inspector3-1129
wip/inspector2-1129
wip/inspector-1129
wip/tagsrev-1110
wip/tags2-1110
wip/tags-1110
job/dates-1148
wip/datesrev-1148
wip/dates2-1148
wip/dates-1148
job/licence-1145
wip/licence2-1145
wip/licence-1145
job/selfhost-1156
job/merge23
wip/merge23
job/tagfilter-1109
wip/tagfilter2-1109
wip/tagfilter-1109
job/kbd-1134
wip/kbd2-1134
wip/kbd-1134
job/palfoot-1137
wip/selfhost-1156
wip/palfoot2-1137
wip/palfoot-1137
job/toggle-1158
wip/toggle-1158
job/kbpreview-1118
job/docratchet-1155
job/perflint-1133
job/devtests-1159
wip/docratchet-1155
wip/devtests-1159
job/segv-1136
wip/toast-1142
wip/segv-1136
job/toast-1142
job/blockreload-1147
wip/blockreload-1147
job/font-1150
wip/font-1150
job/importui-1120
job/minimonth-1149
wip/importui-1120
wip/minimonth-1149
job/depcheck-1146
wip/perflint-1133
wip/depcheck-1146
job/calcards-1115
job/blocks-1125
job/plus-1128
job/shift-1138
wip/plus2-1128
wip/plus-1128
wip/shift-1138
job/moneyfid-1130
job/editorrail-1113
wip/moneyrev-1130
wip/moneyfid-1130
job/noext-851
wip/noext-851
wip/noext3-851
wip/noext2-851
job/week-1135
wip/week-1135
job/editreg-1132
job/smoke-1122
wip/smoke-1122
job/docs-1143
job/palette2-1123
job/calhdr-1112
job/nlpchip-1127
job/mailghost-1094
job/reconnect-1131
wip/reconnect-1131
job/trayicons-1095
job/delete-1119
job/importhang-1121
job/cards-1083
job/palette-1093
job/mchrome-1084
job/e2e-a-1071
job/canvas-visual
job/previewcard-1098
job/allday-1107
wip/e2e-a2-1071
wip/e2e-a-1071
job/e2e-b-1071
job/adv7c-1105
job/kanban-1092
job/agenda-1086
job/merge-round-7c
job/morph-1104
wip/surfaces-p2
job/merge-round-9
wip/merge-round-9
job/7cfix-small
wip/7cfix-small
job/mailui-1078
job/merge-round-8
wip/merge-round-8
wip/mailui-1078
job/mailround-1038
job/applemail-accept
wip/settitle-1068
wip/mailround2-1038
wip/mailround-1038
wip/e2e-7b
job/crash-1069
wip/crash-1069
job/searchlost-1066
wip/searchlost-1066
job/7b-reconcile
job/flake-1065
wip/flake-1065
wip/merge-round-7b7
wip/merge-round-7b6
wip/merge-round-7b5
wip/merge-round-7b4
wip/7b-reconcile
job/appupdate-1059
job/nfd-1044
wip/appupdate-1059
job/e2e-7b
job/loop-1062
wip/loop-1062
job/pdfprev-1045
job/invtoggle-1053
wip/pdfprev-1045
wip/nfd-1044
wip/invtoggle-1053
job/7bfix-e2e
job/mailstress-b
wip/7bfix-e2e
wip/mailstress-b
job/7bfix-adv
wip/7bfix-adv
job/mailstress-a
job/stack-1054
wip/stack-1054
wip/mailstress-a
job/mailstress-1038
wip/mailstress-1038
job/upload500-1051
wip/upload500-1051
job/share-1034
wip/share-1034
job/syncerr-1037
job/7bfix-photos
wip/7bfix-photos
job/paste-1036
job/setside-1039
wip/setside-1039
wip/paste-1036
job/lease-1042
wip/syncerr-1037
wip/lease-1042
job/7bfix-data
job/passkeybind-1043
wip/apprevoke-1041
job/invite-1035
wip/invite-1035
job/merge-round-7b2
wip/merge-round-7b2
job/mailproxy-486
job/apprevoke-1041
job/rebuild-1033
job/pillborder-1029
wip/pillborder-1029
wip/mailproxy-486
wip/applemail-486
job/headless-998
wip/headless-998
job/groups-1028
wip/groups-1028
job/rebuildwarn-1016
wip/rebuildwarn-1016
job/startup-1011
wip/startup-1011
job/monthpill-1009
job/bgthumb-1025
job/sharetitle-1012
wip/monthpill-1009
wip/bgthumb-1025
wip/sharetitle-1012
job/canvas-cards-977
wip/canvas-cards-977
job/canvas-pencil-978
job/canvas-sketch-990
wip/canvas-sketch-990
wip/canvas-pencil-978
job/canvas-files-989
wip/canvas-files-989
job/canvas-collab-991
wip/canvas-collab-991
job/weekscroll-1018
wip/weekscroll-1018
wip/canvas-core-976
job/canvas-core-976
job/round-drag
wip/round-drag
job/round-settings
job/browserfix
wip/oapi-974
job/oapi-974
job/hist2-integrate
job/mailhtml-726
wip/mailhtml-726
wip/hist2-integrate
job/moneyfu-984
job/drag-1015
wip/drag-1015
job/rename-1017
wip/rename-1017
job/hist2-api
wip/hist2-api
job/oneacct-1014
wip/oneacct-1014
wip/moneyfu-984
job/hist2-bench
job/hist2-restore
wip/hist2-bench
job/hist2-write
job/hotfix-724
wip/hotfix-724
wip/hist2-write
wip/hist2-restore
job/hist2-store
job/hist2-ui
wip/hist2-ui
wip/hist2-store
job/searchstarve-965
job/shutdown-963
wip/shutdown-963
wip/pubedit-981
job/pubedit-981
job/analytics-973
wip/searchstarve-965
job/authflash-850
job/weeklane-969
job/pvtitle-1004
job/hist-975
wip/authflash-850
job/voicepill-617
wip/pvtitle-1004
job/headring-1003
wip/weeklane-969
wip/voicepill-617
wip/headring-1003
wip/analytics-973
job/agentscope-980
wip/thumbsandbox-988
job/thumbsandbox-988
wip/hist-975
job/links-856
wip/links-856
job/davetag-966
wip/davetag-966
job/filesstorm-1000
job/hoverpad-725
wip/filesstorm-1000
job/ffmpegblas-993
job/merge-round-7a
wip/hoverpad-725
wip/ffmpegblas-993
job/nowdot-1002
wip/verify-7a
job/noteid-857
wip/nowdot-1002
wip/noteid-857
wip/merge-round-7a
wip/agentscope-980
job/imapedge
job/a11yfix2
wip/imapedge-941
wip/imapedge
wip/a11yfix2
job/notetask-986
job/logheading
wip/logheading-998
job/textthumb-652
job/photolive-987
wip/photolive-987
job/davactive-983
job/savefix-985
job/tabicons-607
wip/davactive-983
wip/tabicons-607
wip/notetask-986
wip/savefix-985
job/dirid-627
job/buildspeed-1007
wip/dirid-627
job/agenda-decks
job/perfguards-impl
job/undo-a11y
wip/undo-a11y
job/mailperf
job/wal-824
wip/settings-50
job/settings-50
job/notesfilter-606
wip/notesfilter-606
job/surfaces-p2
wip/wal-824
job/maillayouts
wip/mailperf
wip/maillayouts
job/taskmeta-659
job/money-ident
wip/money-ident
wip/taskmeta-659
job/errstates
wip/perfguards-impl
job/headings-881
wip/headings-881
wip/errstates
job/voice-619
job/gaps-827
job/notesperf
wip/notesperf
wip/voice-619
job/hddsql-549
job/perf-stream-668
wip/perf-stream-668
wip/deeplinks-fix
job/deeplinks-fix
job/authfix
job/docsfix-rust
wip/docsfix-rust
job/webperf
job/docsfix-web
job/datafix2
job/webdav-lock-476
job/copyfix
wip/copyfix
wip/webperf
job/focus-658
wip/protofix
job/mediafix
job/protofix
wip/mediafix
job/agentfix
job/hhmm-724
wip/agentfix
job/undo-722
job/reuse
wip/webdav-lock-476
wip/reuse
job/scopefix
job/datafix
wip/hhmm-724
wip/undo-722
job/surfaces-p1
wip/hddsql-549
job/voicememos-618
wip/datafix2
wip/surfaces-p1
job/fix-940
wip/fix-940
job/blaze-surfaces
wip/datafix
wip/blaze-surfaces
job/taskday-655
job/linknav-639
wip/linknav-639
wip/gaps-827
job/isolation-707
job/audiophotos-720
wip/audiophotos-720
job/advfind-664
wip/voicememos-618
wip/taskday-655
wip/isolation-707
wip/advfind-664
wip/scopefix
wip/focus-658
job/testgaps
wip/testgaps
job/overscroll-718
wip/authfix
job/deps
wip/overscroll-718
job/rev2-agentfix
job/rev2-money-ident
job/rev2-mailperf
wip/deps
job/hardening-728
wip/hardening-728
job/searchgen-832
wip/searchgen-832
job/photopw-849
job/mailsql-825
wip/photopw-849
job/sharefix
wip/sharefix
job/rev2-mailhtml-726
job/rev2-perfguards
job/copyval-723
job/lightglass-r2
wip/lightglass-r2
wip/docsfix-web
job/copy-audit
job/macinterop-staging-r2
job/design-sync
job/rev2-taskmeta-659
job/rev2-webperf
job/docs-audit
job/rev2-advfind-664
job/rev2-mailproxy-486
job/states-audit
job/rev2-datafix
job/design-drift
job/test-gaps
job/rev2-voicememos-618
job/rev2-mediafix
job/rev2-deps
job/rev2-datafix2
job/licence-audit
job/issue-hygiene
job/rev2-protofix
job/rev2-voice-619
job/rev2-isolation-707
job/rev2-surfaces-p1
job/deeplink-audit2
job/rev2-audiophotos-720
wip/test-gaps
job/rev2-overscroll-718
job/rev2-undo-722
wip/states-audit
job/rev2-dropmd-719
job/rev2-linknav-639
job/merge-7b-plan
wip/merge-7b-plan
job/rev2-taskday-655
wip/mailsql-825
job/rev2-webdav-lock-476
job/rev2-browserfix
wip/design-drift
job/rev2-hddsql-549
wip/deeplink-audit2
job/rev2-scopefix
job/rev2-authfix
job/rev2-hardening-728
job/rev2-wal-824
job/rev2-sharefix
job/calsidebar-638
job/chrome-audit
job/ioperf
wip/ioperf
wip/chrome-audit
wip/calsidebar-638
job/dropmd-719
wip/dropmd-719
job/ocr-build
wip/ocr-build
job/blaze-settings
wip/copyval-723
job/toastring-721
wip/toastring-721
job/deployfix-732
wip/deployfix-732
wip/blaze-settings
job/money-import-recheck
job/rev-a11y
job/perf-arch-db
job/rev-7b-data
wip/textthumb-652
wip/perf-arch-db
job/sec-protocols
job/sidehdr-660
job/rev-7b-security
job/research-surfaces
job/rev-design-gaps
job/rev-mcp-api
wip/sidehdr-660
job/perf-arch-memory
wip/sec-protocols
job/perf-arch-bundle
job/snapedge-714
wip/rev-mcp-api
job/sec-supplychain
wip/research-surfaces
job/perf-arch-sync
job/rev-consistency
job/perf-arch-server
wip/perf-arch-server
wip/perf-arch-memory
job/perf-arch-io
job/perf-arch-client
job/sec-fs
job/sec-mcp-scopes
job/sec-sharing
job/perf-guards
job/sec-browser
job/sec-admin-deploy
job/sec-auth
wip/snapedge-714
job/bgpicker-717
wip/perf-arch-bundle
wip/money-import-recheck
job/advsetup-654
wip/bgpicker-717
wip/advsetup-654
job/burst-709
job/kbdcaps-710
job/app-pw-chooser
wip/burst-709
wip/app-pw-chooser
job/imaptest-625
wip/kbdcaps-710
job/fix-499
wip/fix-499
job/perf-mut-667
job/calimg-589
job/perf-snap-666
wip/calimg-589
wip/perf-snap-666
wip/perf-mut-667
job/perf-cache-665
wip/perf-cache-665
job/voicefiles-620
wip/voicefiles-620
job/admin-burst-705
wip/admin-burst-705
job/voicememos-review
wip/voicememos-review
wip/ryw-653
job/ryw-653
job/writeonopen-661
job/instant-663
wip/writeonopen-661
job/money-import-review
wip/money-import-review
wip/importjs-610
review/integrations-407-round6
wip/integrations-review
job/dragghost-612
wip/dragghost-612
job/integrations
wip/integrations
job/decider-656
job/merge-round-6
job/perf-rerun
wip/merge-round-6
job/integrations-review-round5
job/selalign-576
wip/selalign-576
job/mcp-events-491
job/files-631
job/cal-e2e-569
wip/cal-e2e-569
job/reload-423
wip/reload-423
wip/mcp-events-491
wip/files-631
job/notesbridge-644
wip/notesbridge-644
job/editor-series
job/calcard-series
wip/calcard-series
job/mcp-events-review-491
wip/mcp-events-review
wip/editor-series
job/quirks-546
job/integrations-recheck
job/tocrail-636
wip/tocrail-636
wip/quirks-546
wip/reminders-643
job/reminders-643
wip/davscale-573
job/davscale-573
job/integrations-review
wip/ocr-eval-584
job/ocr-eval-584
job/esc-537
wip/esc-537
job/toastname-586
wip/toastname-586
job/submenu-579
wip/submenu-579
job/tasks-mode
wip/tasks-mode
job/agentdocs-630
job/dupwrite-634
wip/agentdocs-630
wip/dupwrite-634
job/lightglass-588
wip/lightglass-588
job/tabswitch-549
job/ghosttask-623
wip/ghosttask-623
job/toaststack-616
job/weekstate-609
job/mailsync-613
wip/mailsync-613
wip/weekstate-609
job/maildup-626
wip/tabswitch-549
wip/maildup-626
wip/toaststack-616
job/motion-611
wip/motion-611
job/tlstest-601
wip/tlstest-601
job/perf-495
job/floating-sheet
wip/floating-sheet
job/remdup-585
wip/remdup-585
job/fix-502
wip/fix-502
job/attachplay-622
job/perf-batch
wip/perf-batch-563
wip/perf-495
hotfix/mail-sync-diag
job/mail-m3
wip/mail-m3
job/attach-poof-603
job/calhover-608
job/editorbar-604
job/mentions-605
job/merge-round-4
job/allday-514
wip/merge-round-4
wip/allday-514
job/merge-round-4a
wip/merge-round-4a
job/sharestack-580
job/fix-501
wip/sharestack-580
wip/fix-501
job/perf-batch-563
job/apw-cache-review
wip/apw-cache-review
job/probe-520
wip/probe-520
job/mac-393
wip/mac-393
job/header-571
job/flake-513
wip/flake-513
job/docs-thumb-547
wip/header-571
job/webcal-572
wip/webcal-572
wip/shortcuts-542
job/shortcuts-542
wip/docs-thumb-547
job/caldav-stress
wip/caldav-stress
wip/sweep-478
job/apw-cache-512
wip/apw-cache-512
job/money-empty-540
wip/restart-505
wip/money-empty-540
wip/fix-510
job/restart-505
job/fix-503
job/perf-496
wip/perf-496
job/fix-498
wip/fix-498
job/info-inspector-465
wip/info-inspector-465
job/fix-510
job/fix-507
wip/fix-507
wip/fix-503
job/fix-493
job/money-kinds
wip/money-kinds
job/hygiene-548
job/merge-round-3
wip/fix-493
job/drag-snap-536
wip/merge-round-3
wip/merge-round-0930
wip/drag-snap-536
job/align-538
wip/align-538
job/bg-flash
wip/bg-flash
job/money-import
job/search-count-544
wip/search-count-544
wip/money-import
job/settings-key-541
wip/settings-key-541
job/toast-539
job/preview-421
wip/preview-421
wip/toast-539
job/tasks-500-531
job/title-plain-526
wip/title-plain-526
wip/tasks-500-531
job/notes-bridge
wip/parity-484
job/parity-484
job/files-slow
job/crash-525
wip/notes-bridge
wip/files-slow
wip/crash-525
job/kbd-motion-527
wip/bg-422
job/analytics-504
wip/analytics-504
wip/kbd-motion-527
job/upload-pill-523
wip/upload-pill-523
wip/tray-order
job/tray-order
wip/overflow-mid
wip/merge-round-2
job/perf-494
wip/perf-494
wip/mcp-fast-492
wip/motion-477
wip/asr-ab-489
wip/theme-variants-506
wip/overflow-511
wip/week-header-508
wip/attach-427
job/dav-delete-471
job/iso-435
wip/iso-435
wip/files-sel-keys
wip/dav-delete-471
job/align-253
job/siwc-490
wip/siwc-490
job/money-kinds-review
wip/align-253
wip/money-kinds-review
job/small-bugs-3
wip/overlay-title-487
wip/multiget-500
wip/hidden-420
wip/webcal-ui
wip/webcal-431
job/perf-367
job/location
wip/small-bugs-3
wip/location
wip/perf-367
wip/admin-deny-483
job/tag-unicode-473
wip/tag-unicode-473
job/blur-436
wip/photos-470
wip/blur-436
wip/small-bugs-4
wip/hunt-20260930
wip/settings-hdr-482
wip/chips-416
job/dedup-375
wip/dedup-375
job/doc-stack
wip/doc-stack
job/tokens-literals
wip/tokens-literals
job/jobs-leftovers
wip/send-fast
wip/paste-467
wip/money-numbers
job/money-plugin
wip/money-plugin
job/break-dav
wip/merge-batch
wip/crossday-469
wip/mac-verify
wip/mail-m2
wip/break-dav
wip/money-review2
job/money-md
job/modes-424
wip/money-md
wip/jobs-leftovers
job/agenda-413
wip/agenda-413
wip/modes-424
job/recog-417
wip/recog-417
wip/bounce-425
wip/ab-384-luna
job/webdav-perf
wip/webdav-perf
job/toast-ring
wip/toast-ring
job/money-review
wip/money-review
wip/micro-motion
wip/settings-card
wip/minical
job/notes-imap-428
job/least-priv
wip/ui-small-2
wip/flaky-426
wip/drag-end-418
job/jank
wip/jank
wip/least-priv
wip/docs-site
job/agenda
job/sec-batch
wip/sec-batch
wip/per-user-index
job/area-calendars
wip/area-calendars
job/parity
wip/parity
job/documents-research
wip/documents-research
job/test-infra
job/reminders-sync
wip/small-bugs-2
wip/reminders-sync
wip/gestures
job/google-oauth
wip/tags-merge
wip/tags
job/e2e-theme
wip/e2e-theme
job/icon-align
wip/test-infra
wip/select-align
wip/editor-385
job/voice
wip/webdav
job/webdav
job/app-pw-ui
job/editor-integrity
wip/editor-integrity
wip/voice
wip/quota
wip/cal-followups
wip/icon-align
job/composer-scale
wip/composer-scale
job/jobs-page
wip/jobs-page
job/hig-type
wip/hig-type
wip/app-pw-ui
job/motion-spring
job/mcp
wip/motion-spring
wip/mcp
job/small-bugs
wip/push-hosts
job/profile-sign
wip/touch-369
wip/profile-sign
job/mobile-focus
wip/mobile-focus
wip/ui-polish-354
wip/small-bugs
wip/dup-task
job/toast-polish
job/app-pw-scopes
wip/toast-polish
wip/app-pw-scopes
wip/cli-agent
wip/selection-pills
job/preview-attach
wip/preview-attach
job/dav-proppatch
wip/dav-proppatch
wip/cal-switcher
job/atomic-race
wip/atomic-race
job/photos-shared
wip/photos-shared
wip/cal-grid
wip/note-rewrite
wip/search-rebuild
job/mail-m1
job/paperless-import
wip/paperless-import
wip/mail-m1
wip/hidden-activity
wip/search-d
wip/pricing-research
wip/cursors
wip/auto-scheme
job/single-pills
wip/single-pills
wip/xuser-matrix
wip/money-format
wip/app-pw-setup
wip/purge-dos
wip/vault-health
wip/caldav-apple
wip/xuser-audit
wip/e2e-green
wip/tabbar
wip/adv-harness
wip/maple-mono
job/search-fix
wip/search-fix
wip/search-perf-c
job/adv-harness
wip/sidebar-headers
job/glass
wip/temp-index
job/polish
wip/polish
wip/file-protocols
wip/money-research
wip/glass
wip/voice-models
wip/collab-redo
job/voice-research
wip/hunt-20260928
wip/notes-actions-research
wip/search-pad
wip/search-perf
wip/search-sticky
wip/editor-undo
wip/chrome-rules
wip/motion
wip/appearance-research
wip/appearance
wip/audit-bugs
wip/cal-glass
wip/block-actions
wip/authz-order
wip/event-stripes
wip/chrome-sidebar
wip/auth-flaky
wip/robust-2
wip/gate-fix
wip/menu-blur
wip/import-calternaljs
wip/tray-fix
job/import-calternaljs
wip/index-order
wip/audit-fixes
wip/search-chevrons
research/mail
wip/phone-chrome
wip/dedup-break
wip/csp
wip/ui-audit
wip/select-toast
wip/perf
wip/flat-layout
wip/fonts
wip/event-tint
wip/sync-converge
wip/data-split
wip/glass-audit
wip/robustness
wip/sync-chaos
wip/search-thumbs
wip/fuzz
wip/menu-icons
wip/search-pill
wip/sync-changing
wip/heading-links
wip/date-formats
wip/a11y
wip/break-editor
wip/e2e-fix
wip/settings-sections
wip/sync-root-guard
wip/search-palette
wip/share-edit
job/toasts
wip/toasts
wip/cont-analytics
wip/authz-review
wip/popovers
wip/overlay-glass
wip/change-feed
wip/editor-modes
wip/composer-align
wip/cont-agenda
wip/agenda-merge
job/agent-conventions
wip/agent-conventions
wip/backend-misc
job/route-audit
wip/route-audit
wip/ui-batch
wip/heif-hardening
wip/grid-resize
wip/ask-page
wip/webmcp
job/deeplink-audit
wip/deeplinks
wip/shortcuts
wip/cont-tz-days
main
No results found.
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".
No due date set.
Dependencies
No dependencies set
Reference
kayg/calternal#863
Loading…
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Owner rule: every module and every non-obvious function has an up-to-date doc comment that explains the reasoning and invariants; stale comments are defects. Scan crates/**/src and apps/web/src, packages/*/src. Report: modules without a module doc; public functions with non-obvious behaviour and no doc; comments that contradict the code (check a sample deeply: comments naming numbers, limits, issue numbers, file paths, behaviour); comments that name third-party products (not allowed: the repo is public). Group into issues per crate/package.
Read-only job (LIGHT class)
This job runs at the lowest CPU priority on a shared, overloaded build host. Do not build or test: no
cargo build/test/clippy/check, nobun install/build/test/run check, no servers, no browsers. Userg,sed,git,scripts/fj, and reading. Write findings toaudit-findings.mdat the worktree root as you go (append per finding) and commit it on your branch. For each real defect: search existing issues first (scripts/fj --host https://git.kayg.org issue search --repo kayg/calternal "<terms>"), then either add evidence to the existing issue or create one self-contained issue (title, evidence with file:line, owner rule or DESIGN section, expected behaviour, test idea) that a later build job can own. Group findings that share one fix into one issue (one owner per shared fix). No product code changes. Post a summary with the list of issues on your own issue. Docs use ASD-STE100 Simplified Technical English.Starting issue #863 audit in branch
job/docs-audit, based on440e19dce23040ac8ebaae88f0469b6535b1afcb(origin/dev). I am scanning module docs, undocumented public functions with non-obvious behavior, stale comments, and third-party product names across the requested source trees. I will record confirmed findings inaudit-findings.md, search for existing issues, and group any new issues by crate or package.Evidence from Chrome audit #862 (2026-10-02):
apps/web/src/routes/calendar/[view]/[date]/+page.svelte:312-315says Calendar view pills remain in a wide, collapsed-sidebar header. ItscalendarToolssnippet at lines 2036-2038 renders onlyCalendarTools, the date pager. The Calendar view links render inapps/web/src/lib/components/app-sidebar.svelte:595-601.Update the comment to say that Calendar sub-views live in the sidebar and Cmd+K, as DESIGN §34 says.
Copy audit evidence for the third-party-name rule in this issue. These are comments or doc comments in the two UI packages in the #860 scope. This list does not include user-visible strings already reported on #407, #849 or the new screen issues.
Proposed wording: replace a product comparison with the behaviour or platform feature the code relies on. For example, use “native file manager” for Finder, “block editor” for Craft or Obsidian, “photo library” for photo app references, “online photo source” for Unsplash, and “browser engine support” for browser names. For agent stream comments, say “the first agent’s JSON stream” and “the second agent’s JSON stream”.
packages/ui/src:floating.ts:15-16names Chromium, Firefox and Safari/WebKit;floating.ts:269names Finder’s Get Info.components/gestures.ts:159names iOS Safari.tokens.css:740,771,879names Safari.components/FloatingSidebar.svelte:5says “Tahoe-Finder”.components/Checkbox.svelte:11andcomponents/Toggle.svelte:5name SvelteBits.components/files/FileCollection.svelte:26,330,389names Finder.components/viewer/QuickLook.svelte:38names Finder.components/menu/MenuSeparator.svelte:6names Finder and Craft.components/notes/NoteProperties.svelte:42,153names Craft and Obsidian.components/calendar/TimeGrid.svelte:671,673,678names Safari.components/viewer/VideoView.svelte:6names Safari.components/Kbd.svelte:55names Safari and Chrome.components/SegmentedControl.svelte:75,87names Safari.components/menu/FloatingSurface.svelte:440names Safari.weekStart.ts:36-37,139names Firefox, Chromium and Safari.components/calendar/zoom.ts:106names Firefox.components/DateStrip.svelte:10names Chromium.components/ProgressiveBlur.svelte:35names Safari.apps/web/src:lib/appearance/readability.ts:82names Chromium; line 143 names Unsplash.lib/platform.ts:4,lib/themeColor.ts:15,lib/a11y/focusWithKeyboard.ts:5,lib/a11y/inputModality.ts:8,79,90,lib/notifications/push.ts:32,lib/auth/components/CreateAccountFlow.svelte:17,lib/actions/longPressMenu.ts:3,lib/files/uploads.svelte.ts:585,lib/shortcuts/settingsShortcut.ts:20,lib/overlay/scrollLock.ts:4,lib/styles/background.css:203,lib/styles/auth-ui.css:101androutes/settings/notifications/NotificationsSection.svelte:14name Safari.lib/platform.ts:4andlib/shortcuts/format.ts:132-133also name Firefox.lib/a11y/inputModality.ts:79,90names Chrome.lib/photos/PhotoTimeline.svelte:476names Finder and Google Photos;lib/photos/TimelineScrubber.svelte:3,lib/photos/layout.ts:6andlib/search/justify.ts:4name Immich;lib/search/justify.ts:4also names Google Photos.lib/ai/transcript.ts:11,65,222,244,254names Claude Code and Codex.lib/mail/folders.ts:5,9names Gmail.lib/menuNav.ts:72,lib/files/RecentView.svelte:5,lib/files/FilesBrowser.svelte:2,8,379,540,742,775,873,885,966,1041,1552,lib/files/model.ts:97,181,298,331,lib/files/uploads.svelte.ts:32, andlib/files/selection.ts:2,8,120name Finder.lib/auth/passkeys.ts:183names Chrome;lib/components/analytics/widgets/StatRow.svelte:360names Chromium;lib/components/AppToaster.svelte:302names Firefox;lib/notes/NoteEditorSurface.svelte:279,286,384andlib/actions/magnetic.ts:110name Chromium;lib/composer/recorder.svelte.ts:5-6names Chromium, Firefox and Safari;lib/shortcuts/format.ts:132-133names Chromium, Safari and Firefox.lib/files/transfer.ts:81,lib/shortcuts/registry.ts:149,160,250name Chrome;lib/shortcuts/registry.ts:41,155name Finder; line 155 also names Xcode, and line 161 names Gmail;lib/shortcuts/settingsShortcut.ts:20names Safari and Chrome.lib/styles/scrim.css:35,37andcalternal-app.css:131,735name Safari, Chromium or WebKit.routes/+layout.svelte:304names Chrome;routes/calendar/[view]/[date]/+page.svelte:1218names Apple Calendar;routes/money/[budget]/[month]/+page.svelte:505names YNAB;routes/layout.css:183names Immich and Apple Photos.routes/settings/appearance/UnsplashSheet.svelte:2,4,7,68names Unsplash;routes/settings/ai/AiSection.svelte:5names Claude and Codex;routes/settings/calendars/CalendarsSection.svelte:2-3names iCloud, Fastmail and Nextcloud.Expected behaviour: source comments explain the code invariant, browser capability or interaction. They do not use another product as the explanation. Keep needed version and feature details in plain technical terms.
Test idea: add a source check for third-party app and vendor names in comments under
apps/web/srcandpackages/ui/src. Review each hit before accepting the check so provider IDs, URLs and test-only data do not become false positives.The deep-link audit found a stale route comment:
apps/web/src/routes/search/saved/[id]/+page.svelte:2-10says DESIGN §33 lists/s/<saved-search-id>. Current DESIGN §33 lists/search/saved/<id>and states that/s/<slug>is for public links (docs/DESIGN.md:972-999). The running route matches DESIGN; only the comment is stale.Finished the read-only comment audit.
Branch:
job/docs-auditBase:
440e19dce23040ac8ebaae88f0469b6535b1afcb(origin/devat job start)Head:
5988213fafef71081e63fb10677035017a34bf39Built: an audit report only; no product code changed.
Files:
audit-findings.mdFindings include missing module comments, reviewed non-obvious public functions without function comments, third-party product names in comments, stale stylesheet and local research paths, and one motion contradiction. I added the motion evidence to #611. Numeric samples for job retention, the Files Index bind count, and UI size tokens matched the code.
Package follow-ups: #900, #901, #902, #903, #905, #906, #907, #908, #909, #910, #911, #915, #916, #917, #918, #920, #923, #928, #929, #930, #931, #932, #933.
Gate output:
No build/test gates run (read-only LIGHT job forbids cargo and bun commands).Known gaps: the product-name scan used a targeted list, so it does not prove that every possible product name was found. Syntax scans found many simple accessors; I filed the reviewed non-obvious examples and recorded scan limits in
audit-findings.md. No builds or tests ran, as required.Decisions: none beyond applying the explicit owner comment rule and grouping follow-ups by crate or package.
Started the Rust comment fixes from #863 on
job/docsfix-rustat base440e19dce23040ac8ebaae88f0469b6535b1afcb. Read the repo contract, glossary, design decisions, audit findings, and all listed follow-ups. Scope: comments and doc comments undercrates/**only; one commit per Rust follow-up. No runtime changes. Verification:cargo fmt --checkonly, as requested by this job.Starting #863 web comment fixes on
job/docsfix-web, base440e19dce23040ac8ebaae88f0469b6535b1afcb. Scope: comments only inapps/web/srcandpackages/*/src; one commit per package issue. Verification:cargo fmt --checkand Prettier checks on touched files. No runtime changes or test expectation changes.Completed the Rust-side #863 audit findings on
job/docsfix-rust.Head SHA:
a89d7f6d42ede3dbdf3796d31e1876ad9a51f3b7.Merged
origin/devonce before final verification; its merged SHA isc4faf184df726a9375ae0c13bdfb6018ac2cf57e. The merge changed only DESIGN decisions and had no conflicts. No push, deploy, or issue closure.Built: current module and function documentation, repaired stale reasoning, and protocol/format descriptions in place of third-party product names. Executable code, protocol values, test expectations, and dependency versions are unchanged. There are 19 issue commits plus the requested merge.
Findings fixed:
tasks/mod.rsstill said the projector was unwritten; it now explains the current private parser and public re-export boundary.pairing.rsnow holds regression tests;pairing_impl.rsholds the grouping logic. The new module comment describes the actual test boundary.Gate:
cargo fmt --check, exit 0. Verbatim stdout and stderr are empty:Additional source review: all 93 changed Rust files match
origin/devafter removing full-line comments and blank lines. The targeted product-name scan found no matches in 15,993 non-vendored Rust comment lines.git diff --checkpassed. The working tree is clean.cargo cleanremoved the job target output (1 file, 356B). No web build output was created. Optional rustdoc runs were omitted: the shared host load average was 37.52 / 45.56 / 54.74.Known gaps:
Decisions:
UX gaps closed / left: not applicable; no UI or runtime behavior changed.
For the merge round:
cargo run -p calternal-server -- openapi(cd packages/api-client && bun run generate)These must bring
contracts/openapi.jsonand the generated client descriptions into sync with the corrected Rust docs, preserving schema types and API behavior. No adversarial, browser, performance, or runtime test is needed for this comments-only change.Issue commits:
7368c92f4a47f6a136f64b6314864607c9d910cecrates/calternal-auth5254dab95ac98c156b05b7c86da6fdc9dddcc76ecrates/calternal-clia6d0f1be51d58ca6a28e9efa4a08d2a57924e9aecrates/calternal-collab8b70d35e8cd07cc3de616652ffc6e32a93a8b7d3crates/calternal-dav9a2a2b0b08262f3fada91d9c60edda349a9ab461crates/calternal-fs52ad590f8cc15354a15ea2ac80411e0265eec5f9crates/calternal-locationbeb218e832a3e48df3c904b59597f9dfab9ef222crates/calternal-searchc64c1fa44be7cee2c572cc6b9a68899ba0a031c0crates/calternal-imap5e0d7b29fcb630453912ca59105af012b83dd174crates/calternal-money3335ab5fd028c12f4d66657cadfaad1f5e890ad7crates/calternal-notes-core26c07f4d7b76bf87cd9ad221aea41644bd96fee8crates/calternal-server5a1664b469676eb744a0402bed6d389d60b3ea92crates/calternal-sync280eb60f96bd3f780a99f6efc1add05f1f33edc8crates/plugins/aiaf23bb254fe990fe7ed03934f5bc569b9d8e4526crates/plugins/calendardf586c2de8cbffa259d7e0413e12fa93e9a9c81ccrates/plugins/files1578a60c3142b5b18f5c561e65b880d1be2a35a9crates/plugins/mailc9643b1b14751957ca26a3ba81d510d7c92bbcdbcrates/plugins/money580668848f4eab8f8d93fd3d18376f29822f3485crates/plugins/notes7eee92a9b79a44b5d7d09dcc95416d9ca711e8a8crates/plugins/photosFiles:
crates/calternal-auth/src/api.rscrates/calternal-auth/src/profile_signing.rscrates/calternal-auth/src/store.rscrates/calternal-cli/src/main.rscrates/calternal-cli/src/remote_commands.rscrates/calternal-collab/src/lib.rscrates/calternal-collab/src/markdown.rscrates/calternal-collab/src/session.rscrates/calternal-collab/tests/hostile_clients.rscrates/calternal-collab/tests/restart_epoch.rscrates/calternal-collab/tests/wiki_embeds.rscrates/calternal-dav/src/areas.rscrates/calternal-dav/src/files.rscrates/calternal-dav/src/lib.rscrates/calternal-dav/src/protocol.rscrates/calternal-dav/src/reminders.rscrates/calternal-dav/tests/apple_replay.rscrates/calternal-fs/src/file_ops.rscrates/calternal-fs/src/root.rscrates/calternal-fs/src/sidecar.rscrates/calternal-fs/src/trash.rscrates/calternal-fs/src/versions.rscrates/calternal-imap/src/lib.rscrates/calternal-imap/src/mime.rscrates/calternal-imap/src/projection.rscrates/calternal-imap/src/session.rscrates/calternal-imap/src/store.rscrates/calternal-imap/src/submission.rscrates/calternal-imap/src/wire.rscrates/calternal-imap/tests/mime.rscrates/calternal-imap/tests/projection.rscrates/calternal-imap/tests/session.rscrates/calternal-imap/tests/wire.rscrates/calternal-location/src/lib.rscrates/calternal-money/src/budget.rscrates/calternal-money/src/edit.rscrates/calternal-notes-core/src/composer.rscrates/calternal-notes-core/src/lib.rscrates/calternal-notes-core/src/markdown.rscrates/calternal-notes-core/src/nlp/classifier.rscrates/calternal-notes-core/src/nlp/mod.rscrates/calternal-notes-core/src/nlp/recognizers.rscrates/calternal-notes-core/src/nlp/task_types.rscrates/calternal-notes-core/src/nlp/tasks_recurrence_md.rscrates/calternal-notes-core/src/nlp/tasks_sigils.rscrates/calternal-notes-core/src/nlp/tests.rscrates/calternal-notes-core/src/notes_index.rscrates/calternal-notes-core/src/tasks/extract.rscrates/calternal-notes-core/src/tasks/line.rscrates/calternal-notes-core/src/tasks/mod.rscrates/calternal-notes-core/src/tasks/model.rscrates/calternal-search/src/index.rscrates/calternal-search/src/plugin.rscrates/calternal-server/src/appearance.rscrates/calternal-server/src/head.rscrates/calternal-server/src/main.rscrates/calternal-server/src/mcp.rscrates/calternal-server/src/notes_imap.rscrates/calternal-server/src/serve.rscrates/calternal-server/src/wire.rscrates/calternal-sync/src/config.rscrates/calternal-sync/src/credentials.rscrates/calternal-sync/src/engine.rscrates/calternal-sync/src/finder_tags.rscrates/calternal-sync/src/journal.rscrates/calternal-sync/src/lib.rscrates/calternal-sync/src/local.rscrates/calternal-sync/src/remote.rscrates/plugins/ai/src/lib.rscrates/plugins/calendar/src/cache/crypto.rscrates/plugins/calendar/src/client/mod.rscrates/plugins/calendar/src/feeds/publication.rscrates/plugins/files/src/lib.rscrates/plugins/files/src/preferences.rscrates/plugins/files/src/public.rscrates/plugins/files/src/user_bytes.rscrates/plugins/mail/src/cache/store.rscrates/plugins/mail/src/crypto.rscrates/plugins/mail/src/imap.rscrates/plugins/mail/src/sync.rscrates/plugins/money/src/lib.rscrates/plugins/money/src/routes.rscrates/plugins/money/src/views.rscrates/plugins/notes/src/imap.rscrates/plugins/notes/src/lib.rscrates/plugins/notes/src/store.rscrates/plugins/notes/src/tasks_dav.rscrates/plugins/notes/src/tasks_store.rscrates/plugins/notes/tests/apple_replay.rscrates/plugins/photos/src/index.rscrates/plugins/photos/src/migration_tests.rscrates/plugins/photos/src/pairing.rscrates/plugins/photos/src/settings_tests.rsCompleted the comment-only web fixes for #863 on
job/docsfix-web.Built: Added module comments to all 246 files identified by the audit (211 web, 19 UI, 16 editor). Documented the passkey request/ceremony and byte-conversion invariants, the single Settings step-up retry, panel-width persistence/clamping and resize cleanup, and block-anchor parsing/display/matching. Replaced third-party product comparisons with behavior descriptions and corrected stylesheet and local-only research references. Re-read the final module and API comments. Executable source, test expectations, and runner directives are unchanged.
Files: 409 source files: 301 in
apps/web/src, 63 inpackages/ui/src, 45 inpackages/editor/src. The full inventory follows below.packages/api-client/src/generated.tsis unchanged.Commits (one concern per issue):
fe9381d206e0a52cd2b5e58a4b11085446f776e468d99959878cd4245edeb06e25fadfb1fcbc1ceb0bb14c0a05014baea59f1237955a26799ca5b131Head:
a82305864d94a3194015404b6deddbb366ab55b1. Fetched and mergedorigin/devonce before final gates; incorporatedc4faf184df726a9375ae0c13bdfb6018ac2cf57e. The incoming change updated DESIGN §§18, 33, 45 and 57 and was read. No push, deploy or merge into dev/main. Issues remain open.Gates: Only the two checks requested by this job ran; no build, server, browser or runtime test. Cargo used
CARGO_PROFILE_DEV_DEBUG=line-tables-only,CARGO_INCREMENTAL=0,CARGO_BUILD_JOBS=4, and the worktree'starget/tmpfor TMPDIR. CARGO_TARGET_DIR was not overridden. No cargo or web build output was created, so no build cleanup was needed.cargo fmt --check: exit 0; stdout and stderr were empty.bunx prettier --check --plugin <external prettier-plugin-svelte/plugin.js> <409 touched files>: exit 1. Formatter tooling was resolved outside the repo; no dependency files changed. Output verbatim:Formatting baseline comparison with the same formatter: 405 of 409 touched files already fail on
origin/dev, zero new failures, zero changed failure outcomes. Code formatting was left unchanged to honor the comments-only scope.Additional source checks (verbatim):
git diff --check origin/devpassed with no output; the worktree is clean.Known gaps: #903 requires changing source OpenAPI descriptions in Rust and regenerating
contracts/openapi.jsonand the API client together. Those sources are outside this job's ownership; the issue has a handoff comment. No generated-only edit was made. #611 retains the actual GridColumn focus-transition defect; the comment now states the current implementation and its tracking issue. This job cannot change CSS behavior. Existing formatting debt remains. Required environment directives and actual source/API identifier references remain intact.Decisions: No new product/design decision. File-boundary HTML comments describe component modules; TypeScript doc comments describe module/API boundaries. Local-only research references were replaced by the invariant they had explained. Panel docs describe current Installation-local persistence rather than promising a server settings seam.
UX gaps closed: Not applicable: documentation only, no user-facing behavior changed.
UX gaps left: Existing keyboard-motion behavior tracked in #611; no UI change was attempted.
For the merge round: No extra runtime suite is needed for these comment-only changes. #903 needs the Rust-source/contract generation handoff above before that issue can be considered complete.
File inventory:
Repair on
job/docsfix-web(not pushed):bf5d2643fdocs(web): restore platform, browser and library names in comments.Clarified rule (owner, 2026-10-02): do not name competing apps (other notes, calendar, mail or photo apps) in UI, code or comments. Platform, OS, browser, protocol, client and library names that state technical facts stay.
The name scrub replaced facts with garbled stand-ins in about 750 comment lines: "iOS 26.5" became "touch-platform.5", "Safari 15+" became "the browser 15+", "than the" became "tha". This commit restores:
.superpowers/sdd/focus-ring-research.md, research §0/§1/§5b), whatwg/html#8087 with engine versions, WebKit bug 296492 and the iOS gesture research note;Kept: the new module headers, the
app.css→calternal-app.csspath fixes, the #611 motion notes and the retired-task cleanup inslash.test.ts. Competing notes, photo and budget apps (Obsidian, Craft, Notion, Fantastical, Immich, Google Photos, Apple Photos, Apple Notes, YNAB) stay unnamed. Their stand-ins now read as plain English (for example "extended-Markdown pipe syntax", "envelope-budget", "common note editors").Check:
git diff origin/dev...HEAD | grep '^-' | grep -E 'iOS|macOS|Safari|WebKit|Chrom|Firefox|Apple|Finder|Thunderbird|Android|Windows'now lists only moved or rewrapped lines and the competing-app lines. Comments only. Prettier is not set up for Svelte in this repo, so I did not run it.Repair on
job/docsfix-rust(not pushed):3c0ff64e2docs(rust): restore client, platform and library names in comments.Clarified rule (owner, 2026-10-02): do not name competing apps (other notes, calendar, mail or photo apps) in UI, code or comments. Platform, OS, browser, protocol, client and library names that state technical facts stay.
The scrub turned interop facts into vague stand-ins ("Apple Calendar uses a dashed UUID" became "Some CalDAV clients", "Safari on iOS" became "A mobile browser", "Finder" became "file manager") and added a stray "The" to continuation lines. This commit restores Apple Calendar, Reminders, Apple Notes (IMAP), macOS 27, dataaccessd, the #356 and #428 macOS VM and Mac run notes, Finder, Safari, Chromium, iOS, Mac, Fastmail, Aside, Claude Desktop, Claude Code and Codex CLI (
AgentProviderdocs), Unsplash, ProseMirror, Slack, WhatsApp, iMessage, Google feed import rules and Apple MakerNote. The AAE sidecar spelling is now "the iOS and macOS spelling".Kept: the stale-comment fixes from #920, #928, #929, #932 and #933, the version-thinning precision fix and the "Credit Card Payments" / "Inflow: Ready to Assign" names. Competing apps stay unnamed: Obsidian (Tasks), Lightroom, YNAB, Actual Budget and Nextcloud (version thinning).
Gate:
cargo fmt --check: clean (exit 0). Comments only.