Skip to content

Fix the Prologue Extractor command and default the Screenplay model folder - #432

Merged
woksin merged 10 commits into
mainfrom
cli-426-prologue-extractor-command
Oct 20, 2026
Merged

woksin merged 10 commits into
mainfrom
cli-426-prologue-extractor-command

Conversation

@woksin

@woksin woksin commented Oct 20, 2026 •

Copy link
Copy Markdown
Collaborator

Changed

Fixed

  • prologue start now prints an Extractor docker run command that mounts your captures folder into the container, so captures are no longer lost when the container exits, and publishes the OpenTelemetry ports 4317 and 4318 when you capture from OpenTelemetry. The command pins the Extractor image to the bundled Prologue version, and cratis-prologue.json keeps the captures folder you chose, so prologue interpret finds the captures without a path. (prologue start prints an Extractor docker command that loses captures and hides OTLP ports #426)

@woksin woksin added the patch label Oct 20, 2026
@woksin woksin self-assigned this Oct 20, 2026
@woksin

woksin commented Oct 20, 2026

Copy link
Copy Markdown
Collaborator Author

Reviewer notes (not release notes):

  • Combines cli#426, cli#428 and the Screenplay 4.232.0 bump into one release (CI/release economy).
  • Local gate on f4386d4 (after merging origin/main with Stage 4.186.0): dotnet build Cli.slnx -c Release -warnaserror 0 warnings/0 errors; dotnet test Source/Cli.Specs -c Release 4946 passed, 0 failed, 6 skipped (Windows-only). Templating.Specs not run locally (no Templating changes).
  • Review workflow (Opus + Sol): round 1 found 2 blocking issues in the Extractor command (saved captures folder overwritten; image tag 1.0.0 instead of 2.0.0); both fixed, round 2 passed with 0 blocking. Sol confirmed Prologue__Output__Json__Directory against the Prologue 2.0.0 configuration binding.
  • Merge order: merge only after Cratis/Studio#2136 (Screenplay 4.232.0) merges, so the Studio CLI-parity check lines up.
  • Authored as woksin, so no reviewer request; assigned only.

@einari

einari commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

Heads-up: Stage 4.187.1 needs Screenplay >= 4.236.0 (NuGet downgrade rule), so the Stage bump PR for Cratis/Stage#596 moves ScreenplayVersion to 4.236.0 too. If this PR merges first, that one is a one-line rebase of the Screenplay pin.

@woksin woksin changed the title Fix the Prologue Extractor command, default the Screenplay model folder, bundle Screenplay 4.232.0 Fix the Prologue Extractor command and default the Screenplay model folder Oct 20, 2026
@woksin
woksin merged commit 840e4d7 into main Oct 20, 2026
15 of 17 checks passed
@woksin
woksin deleted the cli-426-prologue-extractor-command branch October 20, 2026 06:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants