Skip to content

feat(bench-kit): init safe defaults — pin demo task, install runner deps, prefer https - #34

Merged
psmyrdek merged 1 commit into
masterfrom
feat/bench-kit-safe-defaults
Aug 16, 2026
Merged

feat(bench-kit): init safe defaults — pin demo task, install runner deps, prefer https#34
psmyrdek merged 1 commit into
masterfrom
feat/bench-kit-safe-defaults

Conversation

@psmyrdek

Copy link
Copy Markdown
Collaborator

Wdrożenie punktów A1–A3 z przejścia konsumenckiego template'u 0.7.0 (IDEAS): pierwszy `bench validate` po świeżym `init` nie jest już czerwony z definicji.

Zmiany

  • A1 — pin zadania-demo: init odpalony z wnętrza repo produktowego, poza rejestracją repo w `base_repos`, pinuje zadania-placeholder (`repo: demo-app` + zerowy commit) do wykrytego repo i jego HEAD. Zadania firmowe (bez placeholdera) nigdy nie są dotykane; komentarze YAML przeżywają edycję in-place.
  • A2 — zależności runnera: init i update robią `npm ci` w `.bench-kit/runner` (podmiana strefy przy update kasuje `node_modules` — dotąd pierwsza komenda `bench` po update kończyła się MODULE_NOT_FOUND). Porażka npm degraduje do podpowiedzi, nie blokuje.
  • A3 — preferencja https: wykryty URL SSH jest sondowany po https (`git ls-remote`, `GIT_TERMINAL_PROMPT=0`) i przepisywany, gdy repo odpowiada publicznie — publiczne repo bazowe nie wymaga wtedy żadnych sekretów w CI/kontenerze.

Testy

  • 6 nowych testów (pin demo, preferencja https, npm ci na init i update, degradacja porażki, `toHttpsUrl`); pełna suita 509 zielona, typecheck zielony.

🤖 Generated with Claude Code

… install runner deps, prefer https over SSH

- init pins placeholder tasks (repo: demo-app + all-zeros commit) to the
  detected base repo and its HEAD; company tasks never touched
- init and update run npm ci in .bench-kit/runner (update's wholesale
  swap deletes node_modules); failure degrades to a hint
- detected SSH remote is probed over https (git ls-remote, no terminal
  prompt) and rewritten when the repo answers publicly

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@psmyrdek
psmyrdek force-pushed the feat/bench-kit-safe-defaults branch from 035ffd0 to fcadfae Compare August 16, 2026 17:40
@psmyrdek
psmyrdek merged commit 155bde9 into master Aug 16, 2026
8 of 9 checks passed
@psmyrdek
psmyrdek deleted the feat/bench-kit-safe-defaults branch August 16, 2026 17:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant