Skip to content

fix(ci): don't recurse submodules when bumping the engine pointer - #156

Merged
Seungpyo1007 merged 1 commit into
developfrom
fix/bump-engine-no-recursive
Aug 10, 2026
Merged

fix(ci): don't recurse submodules when bumping the engine pointer#156
Seungpyo1007 merged 1 commit into
developfrom
fix/bump-engine-no-recursive

Conversation

@Seungpyo1007

Copy link
Copy Markdown
Member

Problem

Every bump-engine run has failed since 2026-08-03 — 9 in a row. Before that date, every run succeeded.

TechEngine carries TechAPI back as a submodule, so git submodule update --recursive follows that cycle and re-fetches the million-file data/ tree at every level until the runner's disk is gone:

error: unable to create file data/game/ma/mazzle.json: No space left on device
fatal: Unable to checkout '679e487…' in submodule path
  'TechEngine/TechAPI/TechEngine/TechAPI/TechEngine/TechAPI/TechEngine/TechAPI/TechEngine/TechAPI'

The cycle is not new; what changed on 2026-08-03 is that the job began switching branches (git checkout -B develop origin/develop) before touching the submodule, so it re-resolves it from scratch instead of reusing what actions/checkout had already placed. That was my change, and it turned a latent cycle into a hard failure.

Consequence: the TechEngine submodule pointer has not been bumped automatically for a week. The auto-sync described in .claude/techengine_submodule_sync_spec.md has been silently dead.

Change

Drop --recursive from both submodule commands. The gitlink being bumped is at this repo's top level; nothing below it needs updating, and recursion can only ever walk back into the cycle.

actions/checkout already uses submodules: true (single level), which is correct and unchanged. TechEngine's own workflows use no recursive submodule commands, so nothing is needed on that side.

Refs #1

TechEngine carries TechAPI back as a submodule, so `git submodule
update --recursive` follows that cycle — TechEngine/TechAPI/TechEngine/
TechAPI/… — re-fetching the million-file data tree at every level until
the runner dies with "No space left on device":

    fatal: Unable to checkout '679e487…' in submodule path
    'TechEngine/TechAPI/TechEngine/TechAPI/TechEngine/TechAPI/…'

Every bump-engine run has failed this way since 2026-08-03, which is
when the job started switching branches before touching the submodule
and so began re-resolving it from scratch. The submodule pointer has
not moved automatically since.

The gitlink being bumped is at this repo's top level; nothing below it
needs updating, so drop --recursive from both submodule commands.

Refs #1
@github-actions github-actions Bot added bug Something isn't working ci CI and workflow changes labels Aug 10, 2026
@Seungpyo1007 Seungpyo1007 moved this from Todo to In Progress in TechAPI-Project Aug 10, 2026
@Seungpyo1007
Seungpyo1007 merged commit a707720 into develop Aug 10, 2026
1 check passed
@Seungpyo1007
Seungpyo1007 deleted the fix/bump-engine-no-recursive branch August 10, 2026 08:57
@github-project-automation github-project-automation Bot moved this from In Progress to Done in TechAPI-Project Aug 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working ci CI and workflow changes

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants