ci: don't recurse into nested reference submodules #4

Merged

bjoern merged 1 commit from fix/ci-submodules-non-recursive into main

2026-07-04 20:52:12 +02:00

scarlet commented

2026-07-04 20:50:42 +02:00

Member

Problem

The first main run of the docker-publish workflow failed in the submodule step: git submodule update --init --recursive descended into the packages' own nested reference/ submodules, and novelai_image_gen pins reference/NovelAI.ImageGen on a private GitHub repo the runner has no credentials for:

fatal: could not read Username for 'https://github.com': No such device or address
fatal: Failed to recurse into submodule path 'packages/novelai_image_gen'

Fix

Drop --recursive — only the three top-level package submodules are needed for the image build:

booru_tag_db_dart's openrouter_dart: path: reference/openrouter_dart dependency is never resolved, because angela_server/angela_core dependency_overrides pin openrouter_dart to the top-level submodule.
The previously working Portainer builds used the Dockerfile fallback, which also cloned the three packages non-recursively — empty reference/ dirs are proven fine for dart pub get and the build.

The workflow comment documents why recursion must stay off.

🤖 Generated with Claude Code

## Problem The first `main` run of the docker-publish workflow failed in the submodule step: `git submodule update --init --recursive` descended into the packages' own nested `reference/` submodules, and `novelai_image_gen` pins `reference/NovelAI.ImageGen` on a **private GitHub repo** the runner has no credentials for: ``` fatal: could not read Username for 'https://github.com': No such device or address fatal: Failed to recurse into submodule path 'packages/novelai_image_gen' ``` ## Fix Drop `--recursive` — only the three top-level package submodules are needed for the image build: - `booru_tag_db_dart`'s `openrouter_dart: path: reference/openrouter_dart` dependency is never resolved, because `angela_server`/`angela_core` `dependency_overrides` pin `openrouter_dart` to the top-level submodule. - The previously working Portainer builds used the Dockerfile fallback, which also cloned the three packages non-recursively — empty `reference/` dirs are proven fine for `dart pub get` and the build. The workflow comment documents why recursion must stay off. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

scarlet added 1 commit

2026-07-04 20:50:42 +02:00

ci: don't recurse into nested reference submodules 52c156ef1a

The recursive submodule checkout descended into the packages' own
reference/ submodules; novelai_image_gen pins one on a private GitHub
repo the runner cannot reach, failing the whole step. Only the three
top-level package submodules are needed: booru_tag_db_dart's
reference/openrouter_dart path dep is dependency_override'd to the
top-level submodule, and the previously working Portainer builds also
cloned the packages non-recursively.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>