Skip to content

ci: restart the bridge when lib/host-secrets.ts changes - #91

Merged
EthanThatOneKid merged 1 commit into
mainfrom
deploy-paths
Sep 23, 2026
Merged

EthanThatOneKid merged 1 commit into
mainfrom
deploy-paths

Conversation

@EthanThatOneKid

Copy link
Copy Markdown
Contributor

The bridge imports lib/host-secrets.ts for its credential loading, but that file is missing from the deploy workflow's path filter, so a change to the loader would fast-forward nothing — the workflow would not run at all and the bridge would keep executing stale code. Adds the missing path.

The bridge loads its credentials through lib/host-secrets.ts, but that file is
absent from the deploy workflow's path filter. A fix to the loader therefore
never restarts the bridge: the workflow does not run, the live checkout is not
fast-forwarded, and the running process keeps stale code. Every other module the
bridge imports (discord-bridge, discord-policy, discord-mention-policy,
discord-text) is already listed.
@vercel

vercel Bot commented Sep 23, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
computer Ready Ready Preview Sep 23, 2026 10:59pm UTC

@wazoocomputer

wazoocomputer Bot commented Sep 23, 2026

Copy link
Copy Markdown

This pull request updates the deploy workflow to include lib/host-secrets.ts in its path filters. The change ensures that modifications to the credential loader in lib/host-secrets.ts will trigger the deployment workflow, preventing the bridge from running with stale code.

File Path Change Type Summary
.github/workflows/deploy.yml modified Added lib/host-secrets.ts to path filters.

Reviewers, please focus on the changes in .github/workflows/deploy.yml to ensure the path filter correctly includes the specified file.

1 similar comment
@wazoocomputer

wazoocomputer Bot commented Sep 23, 2026

Copy link
Copy Markdown

This pull request updates the deploy workflow to include lib/host-secrets.ts in its path filters. The change ensures that modifications to the credential loader in lib/host-secrets.ts will trigger the deployment workflow, preventing the bridge from running with stale code.

File Path Change Type Summary
.github/workflows/deploy.yml modified Added lib/host-secrets.ts to path filters.

Reviewers, please focus on the changes in .github/workflows/deploy.yml to ensure the path filter correctly includes the specified file.

@EthanThatOneKid
EthanThatOneKid merged commit e73073b into main Sep 23, 2026
3 checks passed
@EthanThatOneKid
EthanThatOneKid deleted the deploy-paths branch September 25, 2026 21:47

This branch was successfully deployed

1 active deployment
Preview — 6adaeb68 Deployed Sep 23, 2026 by vercel[bot]
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