Commit c667fd5
authored
fix(hermes-plugin): add telemetry credentials generation to CI publish (#1479)
## Summary
- Add `scripts/generate-telemetry-credentials.cjs` to
`apps/memos-local-plugin/` (same script that openclaw already uses)
- Add the "Generate telemetry credentials" step to
`hermes-plugin-publish.yml`, injecting `MEMOS_ARMS_*` secrets so
published npm packages ship with a working `telemetry.credentials.json`
Without this fix, hermes plugin builds have no credentials file and ARMS
telemetry is silently disabled. Both plugins now write to the same ARMS
app, differentiated by the `platform` tag (`"hermes"` vs `"openclaw"`).
## Test plan
- [ ] Verify `hermes-plugin-publish.yml` workflow runs successfully with
the new step
- [ ] Confirm published hermes npm package contains
`telemetry.credentials.json`
- [ ] Validate telemetry events from hermes have `platform: "hermes"` in
ARMS consoleFile tree
2 files changed
+38
-0
lines changed- .github/workflows
- apps/memos-local-plugin/scripts
2 files changed
+38
-0
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
92 | 92 | | |
93 | 93 | | |
94 | 94 | | |
| 95 | + | |
| 96 | + | |
| 97 | + | |
| 98 | + | |
| 99 | + | |
| 100 | + | |
| 101 | + | |
95 | 102 | | |
96 | 103 | | |
97 | 104 | | |
| |||
Lines changed: 31 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
0 commit comments