[ci] Run the App update helper regression suite in CI
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 80/100
Research direction
Start in .github/workflows/ci.yml and pick the job that runs on every push and pull request, such as the L0-L2 static, unit, contract job, checking that its runner has Python 3. Add one step that runs python3 -I -B scripts/docker/app-update-helper.test.py. Run that command locally first to confirm it passes; done means a deliberately broken assertion fails the CI job on a pull request while the unmodified suite passes.
Written by the indexing model from the issue text.
Description
Problem
CI never runs the App update helper's own regression suite, scripts/docker/app-update-helper.test.py. No workflow job, package script or repository gate invokes it; the only Python in .github/workflows/ci.yml is a host snapshot step. The helper stops, renames, restores and starts the running App during a supported update, so a regression in it can leave a user's App stopped, and today only a manual run would catch it.
This gap let a release-blocking defect reach staging during the v0.1.0-rc.1 cycle: an exact-commit update to the commit already running stopped the App, then refused Web publication, then failed automatic restoration because no candidate container existed. The fix added three regressions to this file. They pass when run directly (python3 -I -B scripts/docker/app-update-helper.test.py), but nothing enforces them.
Suggested fix
Add one step that runs python3 -I -B scripts/docker/app-update-helper.test.py to a CI job that runs on every push and pull request, for example the "L0-L2 static, unit, contract" job if its test image has Python 3, or otherwise a runner-level job that uses the runner's system Python. The suite uses only the standard library and in-process fakes, needs no Docker and finishes in about 25 seconds. No new runner, wrapper or dependency is needed. Note that scripts/validate-test-governance.mjs recognizes only JavaScript and TypeScript test files, so no existing gate reports this gap; running the file directly also fails the step if the file is renamed or removed.
Acceptance
A deliberately broken helper assertion fails the CI job on a pull request, and the unmodified suite passes there.
- Dominant language
- TypeScript
- Stars
- 0
- Forks
- 0
- Avg merge
- 12h 53m
- Merged PRs (30d)
- 13
Getting set up
- No Dockerfile or Docker Compose file
- No pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from lingkaix/openkit
-
documentation
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 77/100
Maintainers usually reply within 1 day
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
Maintainers usually reply within 1 day
-
documentation
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
Maintainers usually reply within 1 day
-
Difficulty 4/5 3-5 days Newbie friendliness 45/100
Maintainers usually reply within 1 day
All issues in lingkaix/openkit
Similar issues
-
community first-timers-only good first issue hacktoberfest help wanted low hanging fruit up-for-grabs
Difficulty 1/5 Under an hour Newbie friendliness 90/100
lingdojo/kana-dojo#32663 · 1 comment · 5 reactions ·
Maintainers usually reply within 1 day
-
Admin: Title Separator field can't be cleared (input falls back to "|")Possibly taken @emdashbot claimed this today. Openbot:bug bot:in-review
Difficulty 2/5 1-3 hours Newbie friendliness 80/100
emdash-cms/emdash#4236 · 3 comments ·
Maintainers usually reply within 1 day
-
PHP workflow streaming requests buffer the full SSE responsePossibly taken @Winoren claimed this today. Openreview: medium
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
langgenius/dify#44270 · 1 comment · 1 reaction ·
Maintainers usually reply within 1 day
-
area:holodeck review-finding
Difficulty 2/5 1-3 hours Newbie friendliness 62/100
hyperdrive-technology/holodeck#142 ·
Maintainers usually reply within 1 day
-
Difficulty 1/5 Under an hour Newbie friendliness 90/100
code-yeongyu/senpi#3573 ·
Maintainers usually reply within 1 day