Skip to content

fix(graphite-hub): roll back to sha-84803f5 - #54

Closed
AmrMsCLL wants to merge 1 commit into
mainfrom
fix/hub-rollback
Closed

AmrMsCLL wants to merge 1 commit into
mainfrom
fix/hub-rollback

Conversation

@AmrMsCLL

@AmrMsCLL AmrMsCLL commented Nov 5, 2026

Copy link
Copy Markdown
Member

The sha-1498718 image never became ready: the server imports app/settings.ts and app/share.ts, which the runtime image does not copy. Rolling back restores project-graphite.com; the fix goes forward in graphite-hub.

@AmrMsCLL
AmrMsCLL requested a review from a team as a code owner November 5, 2026 16:24
@github-actions github-actions Bot added the size/xs 0-10 changed lines, excluding lockfiles label Nov 5, 2026
@AmrMsCLL AmrMsCLL closed this Nov 5, 2026
AmrMsCLL added a commit to Project-Graphite/graphite-hub that referenced this pull request Nov 5, 2026
…#33)

`sha-1498718` (#32) never became ready on the VPS, and
`project-graphite.com` went down.

**Cause.** The runtime image copies only `server/`, `build/` and
`scripts/rules.mjs`. Since #32, `server/settings.ts` imports
`app/settings.ts` and `server/share.ts` imports `app/share.ts` at
runtime, not only for types. Neither file was in the image, so the
server could not start.

**Fix.** The image now copies `app/`, about 100 KB of source.

**Guard.** A new **Image starts** job builds the image, runs it in
production mode against a Redis service with placeholder credentials,
and waits for `/health/ready`. Lint and unit tests can't see a file
missing from the image; this job can.

**Checked locally.** Built the image and ran it with
`NODE_ENV=production`: `/health/ready` 200, `/` 200, `/share/home.png`
200 with the DejaVu fonts drawing the text.

**Order.** Project-Graphite/platform#54 rolls the hub back to
`sha-84803f5` to restore the site. Merging this PR builds a fixed image,
and CI records its tag over the rollback.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/xs 0-10 changed lines, excluding lockfiles

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant