Skip to content
This repository was archived by the owner on Aug 29, 2026. It is now read-only.

Fix merged source validation - #225

Merged
howarewoo merged 1 commit into
mainfrom
format-merged-main
Aug 25, 2026
Merged

Fix merged source validation#225
howarewoo merged 1 commit into
mainfrom
format-merged-main

Conversation

@howarewoo

@howarewoo howarewoo commented Aug 25, 2026

Copy link
Copy Markdown
Owner

Goal

Restore the merged main branch to a fully valid release state.

Changes

  • Removes three formatter-rejected trailing blank lines introduced by the merged stack.
  • Restores the session-client test imports and fixtures removed while their dependent tests remained in index.test.ts.
  • Rejects oversized base64 image payloads before scanning their entire alphabet, preventing the 5-second protocol-test timeout observed under the full parallel suite.

Verification

Fresh unauthenticated public clone of commit 3b06ad860f6ef6d288095292ec0b1602671151e7:

  • actionlint .github/workflows/dependency-review.yml
  • pnpm install --frozen-lockfile
  • pnpm format:check
  • pnpm lint
  • pnpm lint:lines
  • pnpm typecheck
  • pnpm test
  • pnpm build

Results: workflow syntax passed; format passed; lint 0 errors; file-length gate passed; 15/15 typecheck tasks; 48 root script tests; all workspace test tasks; 9/9 build tasks.

howarewoo commented Aug 25, 2026

Copy link
Copy Markdown
Owner Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@howarewoo
howarewoo force-pushed the format-merged-main branch 2 times, most recently from 430d9f8 to 1ce5f71 Compare August 25, 2026 15:43
@howarewoo howarewoo changed the title Fix merged source formatting Fix merged source validation Aug 25, 2026
@howarewoo
howarewoo changed the base branch from main to graphite-base/225 August 25, 2026 15:47
@howarewoo
howarewoo changed the base branch from graphite-base/225 to public-dependency-review August 25, 2026 15:47
@howarewoo
howarewoo changed the base branch from public-dependency-review to graphite-base/225 August 25, 2026 15:54
@howarewoo
howarewoo changed the base branch from graphite-base/225 to main August 25, 2026 15:55
@howarewoo
howarewoo marked this pull request as ready for review August 25, 2026 15:57
@howarewoo
howarewoo merged commit 3334e4e into main Aug 25, 2026
2 checks passed

Copy link
Copy Markdown
Owner Author

Merge activity

@howarewoo
howarewoo deleted the format-merged-main branch August 25, 2026 16:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant