Conversation
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1. parse_compose_host_port() — parses both string (127.0.0.1:3000:80) and mapping ({published: 3000}) compose port entries
2. collect_compose_host_port_services() — reads the compose file and returns all (host_port, service_name) pairs
3. get_own_compose_running_ports() — runs docker compose ps --format {{.Ports}} to discover ports currently held by this project's own containers (to avoid false positives on redeploys)
4. check_local_host_port_conflicts() — TCP-binds each declared port; if a bind fails AND the port isn't owned by the project's own containers, it's flagged as a real conflict
5. LocalDeploy::deploy() — calls the check before docker compose up; if conflicts are found, it returns a clear error like:
Host port conflict detected before deploy:
• port 3000 (service 'status-panel-web') is already allocated on this host — find the owner with: lsof -nP -iTCP:3000 -sTCP:LISTEN
Stop the conflicting process or change the port in stacker.yml, then retry.
…red for stripe_connect, constant-time Stripe signature verification, rating duplicate race condition fixed, template rating upsert is now atomic, public rating summary no longer mutates DB, public vendor endpoint now uses slug-only lookup
…ith should be fixed now
…ured as the correct Install Service-compatible mapping. install.inputs section to the stacker.yml
mTLS support
Update DEFAULT_VAULT_URL to :8443 for mTLS
…ompose_host_port returns None for the port, and check_local_host_port_conflicts finds no host ports to validate. This avoids the race condition on TcpListener::bind and the CI port 80 conflict entirely
… are now replaced with case-insensitive regex matching directly on content. Test still passes
…/policy rejections, error taxonomy, marketplace trust, pipe cap
… — marketplace_access, P5 — DAG graph logic, P6 — deploy validator, P4 and P5 deferred
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
No description provided.