Skip to content

fix: locate template.scriban independent of process CWD - #33

Merged
StevenTCramer merged 5 commits into
mainfrom
task/005-fix-scheduled-icon-sync-missing-templatescriban
Aug 30, 2026
Merged

fix: locate template.scriban independent of process CWD#33
StevenTCramer merged 5 commits into
mainfrom
task/005-fix-scheduled-icon-sync-missing-templatescriban

Conversation

@StevenTCramer

Copy link
Copy Markdown
Contributor

Summary

Scheduled CI sync (update-icons --push --publish) crashed looking for template.scriban at the repo root. tools/transform now resolves the template from the binary/project directory (same candidate list as timewarp-heroicons) and skips non-svg files instead of aborting Main.

Fixes failed run: https://github.com/TimeWarpEngineering/timewarp-simple-icons/actions/runs/33307775988

Kanban: 005

Why

CI TransformIconsAsync runs dotnet run --project tools/transform/transform.csproj with working directory = repo root. A relative File.ReadAllText("template.scriban") only worked when scripts/transform.ps1 Push-Locationd into tools/transform.

Test plan

  • dotnet run --project tools/transform/transform.csproj --no-launch-profile from repo root and from /tmp — no FileNotFoundException
  • Non-svg fixture skipped; SVG component written
  • dotnet run tools/dev-cli/dev.cs -- build — 0 warnings, 0 errors
  • After merge, scheduled/sync CI can bump 16.27.1 → latest simple-icons (not run here; no --push --publish)

Notes

ganda task work open-pr hardcodes --base master; this repo’s default branch is main, so the host create failed. This PR was opened with --base main.

StevenTCramer and others added 5 commits August 30, 2026 21:32
CI schedule fails looking for template.scriban at repo root; kitchen
records the heroicons path-resolution fix for the implementer.
Scheduled CI runs transform with working directory = repo root, so a
relative template.scriban path FileNotFoundException aborted icon sync.
Resolve the template from the binary/project dir (heroicons candidate
list) and skip non-svg files with continue instead of aborting Main.

Co-Authored-By: Grok 4.6 <noreply@x.ai>
Co-Authored-By: Grok 4.6 <noreply@x.ai>
Effort-1 general review raised no issues; trail and clean
disposition live under the host task review/ kitchen.

Co-Authored-By: Grok 4.6 <noreply@x.ai>
@StevenTCramer
StevenTCramer merged commit d046eba into main Aug 30, 2026
1 check passed
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