test: enable test_action_menu_window with fixtures

.claude/CLAUDE.md

@@ -0,0 +1 @@
+../AGENTS.md

.github/workflows/build-firmware.yml

@@ -102,7 +102,7 @@ jobs:
 
       - name: Upload log hash dictionary
         uses: Noelware/[email protected]
-        if: ${{ github.event_name == 'push' }}
+        if: ${{ github.event_name == 'push' && github.repository == 'coredevices/PebbleOS' }}
         with:
           access-key-id: ${{ secrets.LOG_HASH_BUCKET_KEY_ID }}
           secret-key: ${{ secrets.LOG_HASH_BUCKET_SECRET }}

.github/workflows/build-prf.yml

@@ -108,7 +108,7 @@ jobs:
 
       - name: Upload log hash dictionary
         uses: Noelware/[email protected]
-        if: ${{ github.event_name == 'push' }}
+        if: ${{ github.event_name == 'push' && github.repository == 'coredevices/PebbleOS' }}
         with:
           access-key-id: ${{ secrets.LOG_HASH_BUCKET_KEY_ID }}
           secret-key: ${{ secrets.LOG_HASH_BUCKET_SECRET }}

.github/workflows/release.yml

@@ -186,6 +186,7 @@ jobs:
 
       - name: Upload log hash dictionary
         uses: Noelware/[email protected]
+        if: ${{ github.repository == 'coredevices/PebbleOS' }}
         with:
           access-key-id: ${{ secrets.LOG_HASH_BUCKET_KEY_ID }}
           secret-key: ${{ secrets.LOG_HASH_BUCKET_SECRET }}

.gitlint

@@ -4,9 +4,16 @@ ignore-merge-commits=false
 ignore-fixup-commits=false
 ignore-fixup-amend-commits=false
 ignore-squash-commits=false
+regex-style-search=true
 
 [title-match-regex]
-regex=[a-z0-9/]+: .*
+# Format: "area: description" where area is either:
+# - A path with at least one / (e.g., fw/drivers/hrm, third_party/nonfree)
+# - One of the known short areas: ci, treewide, platform, sdk, tools, resources,
+#   docs, waftools, settings, libc, tests, notifications, build, wscript, fw,
+#   third_party, ancs, compositor, console, kernel, health
+# Conventional commit types like feat:, fix:, chore: are NOT allowed.
+regex=^([a-z0-9_]+/[a-z0-9_/]*|ci|treewide|platform|sdk|tools|resources|docs|waftools|settings|libc|tests|notifications|build|wscript|fw|third_party|ancs|compositor|console|kernel|health|gitlint|readme|requirements|python_libs|pbl-tool|pbl|moddable|libutil|iconography|gitignore|capabilities|asterix|activity|accel): .*
 
 [title-max-length]
 line-length=100

AGENTS.md

@@ -0,0 +1,49 @@
+# PebbleOS
+
+PebbleOS is the operating system running on Pebble smartwatches.
+
+## Organization
+
+- `docs`: project documentation
+- `python_libs`: tools used in multiple areas, e.g. log dehashing, console, etc.
+- `resources`: firmware resources (icons, fonts, etc.)
+- `sdk`: application SDK generation files
+- `src`: firmware source
+- `tests`: tests
+- `third_party`: third-party code in git submodules, also includes glue code
+- `tools`: a variety of tools or scripts used in multiple areas, from build
+  system, tests, etc.
+- `waftools`: scripts used by the build system
+
+## Code style
+
+- clang-format for C code
+
+## Firmware development
+
+- Configure: `./waf configure --board BOARD_NAME`
+
+  - Board names can be obtained from `./waf --help`
+  - `--release` enables release mode
+  - `--mfg` enables manufacturing mode
+  - `--qemu` enables QEMU mode
+
+- Build main firmware: `./waf build`
+- Build recovery firmware (PRF): `./waf build_prf`
+- Run tests: `./waf test`
+
+## Git rules
+
+Main rules:
+
+- Commit using `-s` git option, so commits have `Signed-Off-By`
+- Always indicate commit is co-authored by Claude
+- Commit in small chunks, trying to preserve bisectability
+- Commit format is `area: short description`, with longer description in the
+  body if necessary
+- Run `gitlint` on every commit to verify rules are followed
+
+Others:
+
+- If fixing Linear or GitHub issues, include in the commit body a line with
+  `Fixes XXX`, where XXX is the issue number.

src/fw/drivers/hrm/as7000/as7000.c

@@ -514,7 +514,6 @@ static void prv_disable(HRMDevice *dev) {
       WTF;
   }
   led_disable(LEDEnablerHRM);
-  analytics_stopwatch_stop(ANALYTICS_DEVICE_METRIC_HRM_ON_TIME);
 }
 
 // NOTE: the caller must hold the device's state lock
@@ -525,7 +524,6 @@ static void prv_enable(HRMDevice *dev) {
 
   } else if (dev->state->enabled_state == HRMEnabledState_Disabled) {
     led_enable(LEDEnablerHRM);
-    analytics_stopwatch_start(ANALYTICS_DEVICE_METRIC_HRM_ON_TIME, AnalyticsClient_System);
 
     // Enable the device and schedule a timer callback for when we can start communicating with it.
     gpio_output_set(&dev->en_gpio, true);