ci: add Forgejo Actions build pipeline #1

Open

marcy wants to merge 40 commits from forgejo-build-pipeline into main

pull from: forgejo-build-pipeline

merge into: infra.pink:main

infra.pink:main

infra.pink:scenario-management-system

Conversation 0 Commits 40 Files changed 28 +4716 -216

marcy commented 2026-01-16 11:09:12 -08:00

Owner

Copy link

Summary

• Add Forgejo Actions CI/CD workflow for building and publishing Pygmalion
• Test job runs on every push with continue-on-error for alpha builds
• Matrix build job creates self-contained executables for win-x64 and osx-x64
• Upload job publishes packages to Forgejo registry on main branch only

Test Plan

• Push to non-main branch: test and build jobs run, upload job skipped
• Push to main branch: all jobs run, packages uploaded to registry
• Check packages visible in repository's Packages section

## Summary - Add Forgejo Actions CI/CD workflow for building and publishing Pygmalion - Test job runs on every push with continue-on-error for alpha builds - Matrix build job creates self-contained executables for win-x64 and osx-x64 - Upload job publishes packages to Forgejo registry on main branch only ## Test Plan - [ ] Push to non-main branch: test and build jobs run, upload job skipped - [ ] Push to main branch: all jobs run, packages uploaded to registry - [ ] Check packages visible in repository's Packages section

marcy added 37 commits 2026-01-16 11:09:12 -08:00

docs: add editor script & quest improvements design plan ... 03b2e47312

Completed brainstorming session. Design includes:
- Form-based script editor replacing text area
- New quest editor panel with relationship tracking
- Consistent TOML serialization with semantic field ordering
- Cross-panel navigation for quest authoring workflow
- 6 implementation phases

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

feat(editor): add OrderedTomlBuilder for consistent TOML field ordering ca8df9b9fe

feat(editor): add SerializeQuest method for quest TOML output bd3ea7304b

refactor(editor): update SerializeScript to use OrderedTomlBuilder 4ccc54499f

refactor(editor): update SerializeMapSpawns to use OrderedTomlBuilder e2c8b10e18

test(editor): add tests for OrderedTomlBuilder and quest serialization c1d66c81c2

test(editor): verify round-trip consistency for quest serialization ... 973aa8c3cd

- Round-trip test: SerializeQuest_Roundtrip_ProducesIdenticalOutput passes
- Confirms that re-serializing a quest produces identical TOML output
- All 6 Roundtrip tests pass
- Ensures consistent field ordering maintains round-trip semantics

fix: remove unused _orderedKeys field in OrderedTomlBuilder ... 286653ec97

The _orderedKeys field was populated but never read. Since Tomlyn's internal
implementation maintains insertion order automatically (as documented in the
Build() method comment), tracking keys manually serves no functional purpose.

- Removed _orderedKeys field declaration
- Removed all _orderedKeys.Add() calls from Add, AddIf, AddIfNotEmpty, AddIfNotNull methods
- Verified build succeeds with no new errors

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

feat(editor): add EditableScript model for form-based script editing d6b5cf8f9e

feat(editor): redesign ScriptEditorPanel with form-based node editing 59266a8869

test(editor): add tests for EditableScript model 635597ba53

fix: address code review feedback for script editor ... 33171bff76

- Important Issue: Add debug logging to empty catch block in ScriptEditorPanel when handling directory scanning errors (line 762-765)
- Minor Issue 1: Add test case for choice reference updates in UpdateNodeReferences to verify dialogue choices are properly updated when node IDs change
- Minor Issue 2: Implement action editor key renaming by tracking parameter key changes and updating the Params dictionary accordingly

All tests pass (545/545).
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

feat(editor): add EditableQuest model for quest editing 252234219a

feat(editor): add QuestEditorPanel for quest definition editing 2db5f33ced

feat(editor): register QuestEditorPanel in game 47f855178b

test(editor): add tests for EditableQuest model b26f2276c7

feat(editor): add QuestRelationshipScanner for finding quest references ef851a8c47

feat(editor): add relationship tracking display to QuestEditorPanel 3a391bc901

test(editor): add tests for QuestRelationshipScanner 4b95a152ca

feat(editor): add NavigateToQuest and CreateScriptForQuest to EditorContext c35faa11fe

feat(editor): update QuestEditorPanel to handle navigation requests 77d3ca3b25

feat(editor): add CreateScriptForQuest support to ScriptEditorPanel 0b530822c0

feat(editor): handle NavigateToQuest and CreateScriptForQuest in EditorState 816f6c37f5

feat(editor): add navigation from Script Editor to Quest Editor 4435d86d37

test(editor): add tests for editor navigation properties e6cafb58a7

docs: update EDITOR.md with Quest Editor and form-based Script Editor 0d09ade6d0

docs: update SCENARIO_FORMAT.md with TOML field ordering details a44d6efc90

polish(editor): add tooltips for better UX 1b5a2cda72

docs: add Quest Editor to file formats table fa44436450

docs: update project context for editor-script-quest-improvements ... eeff458872

Add Editor System to key systems documentation in both AGENTS.md and
CLAUDE.md. Update testing section to include dotnet test command now
that unit tests exist.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

docs: add Forgejo build pipeline design plan ... 8ee2e85938

Completed brainstorming session. Design includes:
- Matrix build for Windows and macOS in parallel
- Test job with continue-on-failure for alpha builds
- Package upload to Forgejo registry (main branch only)
- 4 implementation phases

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

docs: add Forgejo build pipeline implementation plan ... e80b4736bf

Four-phase implementation plan:
- Phase 1: Workflow scaffolding with three job stubs
- Phase 2: Test job with continue-on-failure
- Phase 3: Matrix build for win-x64 and osx-x64
- Phase 4: Package upload to Forgejo registry (main only)

Includes:
- zip/curl installation for Docker containers
- Prerequisites for Forgejo secrets/variables
- Correct artifact download paths

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

ci: add Forgejo Actions workflow scaffold ... 42bf1f4170

Three-job pipeline structure:
- test: runs after checkout
- build: matrix for win-x64 and osx-x64, depends on test
- upload: conditional on main branch, depends on build

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

ci: implement test job with continue-on-failure ... dfc9ad1df8

- Restore dependencies before testing
- Run dotnet test with --no-restore for efficiency
- Set continue-on-error: true to allow alpha builds even with failing tests

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

ci: implement matrix build job for win-x64 and osx-x64 ... c2c30edd14

- Publish self-contained executable for each platform
- Copy platform-specific fnalibs to publish output
- Create ZIP archive for distribution
- Upload as workflow artifact with 30-day retention

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

ci: implement package upload to Forgejo registry ... 6f4c58c5b7

- Download artifacts from build job
- Compute version as YYYY-MM-DD-<short-sha>
- Upload to Forgejo generic packages via curl PUT
- Only runs on main branch pushes
- Uses FORGEJO_TOKEN secret and FORGEJO_API_URL variable

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

docs: update project context for forgejo-build-pipeline ... f1d635a33e

Add CI/CD section documenting Forgejo Actions workflow with test, build,
and upload jobs. Update freshness dates.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

marcy added 1 commit 2026-01-16 11:40:43 -08:00

ci: install Node.js for GitHub Actions in container jobs ... d963534858

The actions/checkout, actions/upload-artifact, and actions/download-artifact
actions require Node.js to run. The .NET SDK Docker image doesn't include
Node.js, so we need to install it before using these actions.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

marcy added 1 commit 2026-01-16 11:46:38 -08:00

ci: downgrade artifact actions to v3 for Forgejo compatibility ... 723d18eb3a

The v4 versions of upload-artifact and download-artifact use GitHub's
new artifact backend which is not supported on Forgejo/self-hosted runners.
Downgrade to v3 which uses the compatible artifact storage.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

marcy added 1 commit 2026-01-16 12:02:17 -08:00

ci: enable single-file publishing with FNA.dll excluded ... a3872830c6

Per FNA distribution guidelines, enable PublishSingleFile=true for a
single executable output while excluding FNA.dll from bundling. This
allows developers and users to swap the FNA library for debugging or
modding purposes.

- Add ExcludeFromSingleFile to FNA ProjectReference in Engine.csproj
- Add -p:PublishSingleFile=true to dotnet publish in workflow

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

All checks were successful

Hide all checks

Build / test (push) Successful in 1m0s

Details

Build / build (osx, osx-x64) (push) Successful in 2m13s

Details

Build / build (x64, win-x64) (push) Successful in 1m23s

Details

Build / upload (push) Has been skipped

Details

This pull request has changes conflicting with the target branch.

• AGENTS.md
• CLAUDE.md
• src/Pygmalion.Engine.Tests/Unit/Editor/QuestRelationshipScannerTests.cs
• src/Pygmalion.Engine/Editor/Panels/QuestEditorPanel.cs
• src/Pygmalion.Engine/Editor/Panels/ScriptEditorPanel.cs
• src/Pygmalion.Engine/Editor/QuestRelationshipScanner.cs

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin forgejo-build-pipeline:forgejo-build-pipeline

git switch forgejo-build-pipeline

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch main

git merge --no-ff forgejo-build-pipeline

git switch forgejo-build-pipeline

git rebase main

git switch main

git merge --ff-only forgejo-build-pipeline

git switch forgejo-build-pipeline

git rebase main

git switch main

git merge --no-ff forgejo-build-pipeline

git switch main

git merge --squash forgejo-build-pipeline

git switch main

git merge --ff-only forgejo-build-pipeline

git switch main

git merge forgejo-build-pipeline

git push origin main

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

infra.pink/pygmalion!1

No description provided.