Remove stray debug files and stale generated agent copy #78

Closed
opened 2026-10-08 16:33:32 -04:00 by goddard · 1 comment
Collaborator
  • tmp-resp.txt and tmp-dedicated.cfg are leftover debug files.
  • static/agent/ is a generated copy that npm run dev doesn't refresh.

Fix

  • Delete tmp-resp.txt and tmp-dedicated.cfg
  • Regenerate static/agent/ as part of npm run dev (or stop committing it and generate it at build time)
- `tmp-resp.txt` and `tmp-dedicated.cfg` are leftover debug files. - `static/agent/` is a generated copy that `npm run dev` doesn't refresh. **Fix** - [x] Delete `tmp-resp.txt` and `tmp-dedicated.cfg` - [x] Regenerate `static/agent/` as part of `npm run dev` (or stop committing it and generate it at build time)
goddard added the
priority
low
type/maintainability
labels 2026-10-08 16:39:19 -04:00
josh was assigned by goddard 2026-10-08 17:31:51 -04:00
Author
Collaborator

Reviewed e9be3f7177. Both fix items are done, so I'm closing this.

Checklist

  • Debug files deleted. tmp-resp.txt and tmp-dedicated.cfg are gone, and nothing in the repo refers to them.
  • static/agent/ regenerated by npm run dev. predev now runs agent:package first (package.json:25). That writes all four files in static/agent/: the agent, the updater, install.sh and the checksum manifest (package_release.py:26-31). The README and development guide drop the manual npm run agent:package step, and they say to restart dev or rerun it after editing agent code.

Review notes

  • Running npm run build, which packages the agent the same way, left the working tree clean. The committed static/agent/ copy matches agent/ today.
  • Minor and non-blocking: the committed copy can still drift if agent source is committed without running dev or build. A test comparing static/agent/ with agent/ would catch that, and it could go with the CI work in #77.
Reviewed e9be3f7177b2810c2ca688501f360ef24cf29ac1. Both fix items are done, so I'm closing this. **Checklist** - [x] **Debug files deleted.** `tmp-resp.txt` and `tmp-dedicated.cfg` are gone, and nothing in the repo refers to them. - [x] **`static/agent/` regenerated by `npm run dev`.** `predev` now runs `agent:package` first (`package.json:25`). That writes all four files in `static/agent/`: the agent, the updater, `install.sh` and the checksum manifest (`package_release.py:26-31`). The README and development guide drop the manual `npm run agent:package` step, and they say to restart dev or rerun it after editing agent code. **Review notes** - Running `npm run build`, which packages the agent the same way, left the working tree clean. The committed `static/agent/` copy matches `agent/` today. - Minor and non-blocking: the committed copy can still drift if agent source is committed without running dev or build. A test comparing `static/agent/` with `agent/` would catch that, and it could go with the CI work in #77.
Sign in to join this conversation.