Skip to content

Upgrade#25

Merged
jlwaugh merged 5 commits into
MultiAgency:mainfrom
elliotBraem:main
May 18, 2026
Merged

Upgrade#25
jlwaugh merged 5 commits into
MultiAgency:mainfrom
elliotBraem:main

Conversation

@elliotBraem
Copy link
Copy Markdown
Contributor

Makes some updates, see comments

Comment thread .github/workflows/ci.yml

- name: Log in to GitHub Container Registry
uses: docker/login-action@v3
uses: docker/login-action@c94ce9fb468520275223c153574b00df6fe4bcc9 # v3.7.0
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

changes to github actions are security related

pinning version hashes
scoping secrets to proper workflows

Comment thread .github/dependabot.yml
version: 2
updates:
- package-ecosystem: "bun"
directory: "/"
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

removed for renovate, but also -- you don't need to merge the dependency PRs

I'm tempted to remove it entirely from children and just have parent always handle dependency version syncing (since inheritance is the advantage here)

Comment thread .github/renovate.json
},
{
"matchPackagePatterns": ["^@rspack/", "^@rsbuild/"],
"allowedVersions": "<2.0.0"
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Because I noticed you upgraded these to v2, but we're not ready for v2


export function Shell({ children }: { children: ReactNode }) {
const pathname = useClientValue(() => window.location.pathname, "/");
const matchRoute = useMatchRoute();
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

tanstack router idiomatic

Comment thread ui/src/lib/api.ts

function createRpcLink(
runtimeConfig: { hostUrl: string; rpcBase: string },
forwardCookie?: string,
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

this was a workaround, probably cuz rspack was failing so you were trying to get working api

Comment thread ui/src/agency-theme.css
@@ -1,13 +0,0 @@
@import url("https://fonts.googleapis.com/css2?family=Bungee&display=swap");
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

sorry I realized styles.css was getting gitignored

I moved to there, follow standards

Comment thread ui/src/router.server.tsx
const renderToStream = async (request: Request, renderOptions: RenderOptions) => {
const url = new URL(request.url);
const history = createMemoryHistory({ initialEntries: [url.pathname + url.search] });
const { hostUrl, rpcBase } = renderOptions.runtimeConfig;
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

shouldn't need these workarounds

Comment thread .templatesync-exclude
@@ -1,42 +0,0 @@
# Files copied on `bos init` but never overwritten on `bos sync`.
# These are agency-owned — every fork customizes them after init.
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Migrated away from this concept for upgrades

Comment thread bos.config.json
@@ -1,12 +1,48 @@
{
"extends": "bos://dev.everything.near/everything.dev",
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

extending from dev.everything.near

Comment thread bos.config.json
]
}
},
"app": {
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

no need to describe host or auth -- these will stay up to date with parent (dev.everything.near)

@jlwaugh jlwaugh merged commit 961c176 into MultiAgency:main May 18, 2026
2 checks passed
@railway-app railway-app Bot temporarily deployed to dashboard / production May 18, 2026 17:17 Inactive
@jlwaugh jlwaugh deployed to production May 18, 2026 17:17 — with GitHub Actions Active
@railway-app railway-app Bot temporarily deployed to dashboard / staging May 18, 2026 19:28 Inactive
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.

2 participants