Skip to content

docs(readme): pre-alpha disclaimer + honest status#35

Merged
SolshineCode merged 1 commit into
mainfrom
docs/early-alpha-disclaimers
May 12, 2026
Merged

docs(readme): pre-alpha disclaimer + honest status#35
SolshineCode merged 1 commit into
mainfrom
docs/early-alpha-disclaimers

Conversation

@SolshineCode
Copy link
Copy Markdown
Owner

Prominent top-of-README disclaimer that the project is pre-alpha and not a playable game today. Lists verified vs unverified pieces; points at the retraction + Unciv research docs; sets realistic expectations for a fresh clone. Architecture diagram + milestone grid updated to honestly show the save-edit tool as the unimplemented critical path.

Prominent top-of-README disclaimer block stating the project is
pre-alpha and not a playable game today. Lists verified vs not-
verified pieces, points readers at the retraction + Unciv research
docs, sets realistic expectations for cloning today.

Architecture diagram updated to show the save-edit tool as the
unimplemented critical-path piece. Milestone status grid honestly
shows bridge-side ✅ alongside Unciv-side ❌ for M1/M2.

Added a "what this is" footer note explaining the project's
experimental nature and the choice to publish in honest current
state rather than polished-looking-but-broken state.

No code changes. 248 tests still pass.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@SolshineCode SolshineCode merged commit 24a6dcb into main May 12, 2026
@SolshineCode SolshineCode deleted the docs/early-alpha-disclaimers branch May 12, 2026 01:16
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the README.md to accurately reflect the project's pre-alpha status, explicitly stating that the game is not yet playable and detailing the unimplemented integration with Unciv. The changes include updated architecture diagrams, a revised development roadmap, and clearer setup instructions for the TUI sandbox and bridge daemon. Review feedback identifies a filename inconsistency in the architecture diagram and notes the omission of the scripts directory from the repository layout section.

Comment thread README.md
│ │ │
▼ ▼ ▼
Textual TUI ./saves/ [save-edit tool: NOT BUILT]
(live view) kingdom.json │
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The architecture diagram refers to kingdom.json, but the project consistently uses kingdom_save.json elsewhere (e.g., lines 74, 95, and in bridge/save_exchange.py). Updating this will ensure documentation consistency.

Suggested change
(live view) kingdom.json
(live view) kingdom_save.json │

Comment thread README.md
tests/ TUI tests (incl. Pilot)
mod/ClaudeKingdoms/ Unciv mod content (JSON only; unvalidated)
docs/ PRD, ADRs, schema, hooks-setup, audit, research
.github/workflows/ CI (pytest on push/PR)
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The scripts/ directory is missing from the repository layout section, although it is still referenced in the "Testing" section (line 190). It should be included to provide a complete overview of the project structure.

Suggested change
.github/workflows/ CI (pytest on push/PR)
scripts/ Auxiliary scripts (hello-world smoke, hook logger PoC)
.github/workflows/ CI (pytest on push/PR)

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