Modernize Phorge development environment and simplify workflow #120
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Rename Containerfile.phorge to Dockerfile.phorge for better tooling support - Rename compose-phorge.yml to compose.yml (standard Docker Compose name) - Remove legacy docker-compose-phabricator.yml (Phabricator support deprecated) - Consolidate AUTOMATED_SETUP.md into phorge-setup.md for cleaner docs - Rename env vars from PHABRICATOR_* to PHORGE_* for consistency - Extract inline PHP config to phorge.ini file - Make setup scripts configurable via environment variables - Improve documentation formatting and clarity Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add PHORGE_ADMIN_PASS environment variable for setting passwords directly - Implement password hashing matching Phorge's HMAC-SHA256 + bcrypt scheme - Set passwords for admin and all test users during container startup - Skip recovery link generation when password is set directly - Update documentation with new configuration option Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Make all Phorge settings overridable via environment variables - Add PHORGE_GIT_REF and ARCANIST_GIT_REF to checkout specific branches/tags - Rename PHORGE_ALT_FILE_DOMAIN to PHORGE_CDN_URL - Consolidate Makefile variables with compose.yml (extract hostnames from URLs) - Update docs with environment variables table and command-line examples - Print git history links instead of just commit hashes at startup Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Rename phorge-{up,down,logs,shell} to {up,down,logs,shell}
- Replace phabfive-run/phabfive-run-dev with phorge/phabfive wrapper script
- Rename phabfive-build to image and move to Build section
- Move format target to Essential Development section
- Add terminfo to Dockerfile for proper terminal support
- Update docs to emphasize `uv run phabfive` as primary approach
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Show task hierarchy with indentation instead of 66 CRITICAL messages - Display task titles that would be created - Single warning message at start instead of per-task spam - Move mega-2024-simulation.yml to templates/task-create/ Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- lock: sync uv.lock with pyproject.toml - upgrade: upgrade all deps to latest compatible versions Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Updated cachetools v6.2.0 -> v6.2.4 Updated certifi v2025.10.5 -> v2026.1.4 Updated charset-normalizer v3.4.3 -> v3.4.4 Updated click v8.3.0 -> v8.3.1 Updated coverage v7.10.7 -> v7.13.1 Updated exceptiongroup v1.3.0 -> v1.3.1 Updated filelock v3.20.0 -> v3.20.3 Updated idna v3.10 -> v3.11 Updated iniconfig v2.1.0 -> v2.3.0 Updated markdown v3.9 -> v3.10 Updated pathspec v0.12.1 -> v1.0.3 Updated platformdirs v4.5.0 -> v4.5.1 Updated pytest v8.4.2 -> v9.0.2 Updated ruamel-yaml v0.18.16 -> v0.19.1 Removed ruamel-yaml-clib v0.2.14 Updated tox v4.31.0 -> v4.34.1 Updated urllib3 v2.5.0 -> v2.6.3 Updated uv v0.9.1 -> v0.9.24 Updated virtualenv v20.35.1 -> v20.36.1
holmboe
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Dockerfile.phorge,compose.yml)PHORGE_ADMIN_PASSfor immediate login without recovery linkPHORGE_GIT_REFandARCANIST_GIT_REFto specify git branchesup,down,logs,shellinstead ofphorge-*)phorge/phabfivewrapper script for Docker-based runslockandupgradetargets for dependency management--dry-runoutput to show task hierarchy instead of spamtemplates/task-create/Test plan
make upand verify container starts correctlyPHORGE_GIT_REF=master make up)PHORGE_ADMIN_PASSenables immediate login./phorge/phabfive user whoamiworks--dry-runshows task tree correctlymake lockandmake upgradework🤖 Generated with Claude Code