fix: capture large brand pages and add Xquik preset - #103
Open
kriptoburak wants to merge 1 commit into
Open
Conversation
kriptoburak
force-pushed
the
codex/fix-brand-capture-large-html
branch
from
August 26, 2026 23:09
6da12ab to
5af0f40
Compare
kriptoburak
force-pushed
the
codex/fix-brand-capture-large-html
branch
from
August 27, 2026 06:38
5af0f40 to
a2844a6
Compare
Author
|
Resolved the live
The PR is now one commit over |
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
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.
Problem and value
Fixes #102.
brands captureread the whole HTML response against its 256 KiB cap before inspecting icon metadata. A site with a small head and a large body failed before Archify reached valid icon candidates.This change also adds Xquik to the maintained Archify brand catalogue. Users can resolve
xquik,xquik-api,xquik.com, ordocs.xquik.com. Archify renders the bundled public vector without a network capture.Scope
</head>.Stability impact
Current-base repair
aab9f85659ada148a9232a5129726e395edb4370.archify.zipfrom the combined source.Tests run
node --test test/brand-marks.test.mjs: 28 passed, 0 failed.npm test: 719 passed, 0 failed, 18 correctly skipped.archify.zippackage smoke: passed on macOS.archify.zipbyte-for-byte.npm run check:brand-marks: passed.git diff --check upstream/main...HEAD: passed.The focused regression splits
</head>across 2 chunks, then sends a body over 300 KiB. A second regression confirms an oversized head still fails. The Xquik regression covers domain lookup, preset capture, provenance, rendered SVG metadata, and the installed package.Visual evidence
Real Chrome
visual-checkpassed at 1440×900, 1600×1000, 1920×1080, and 2048×1320. The Xquik mark stays visible and clear of labels, routes, the legend, and viewer controls.Generated artifacts
Rebuilt
archify.zipfrom the current source after rebasing ontomain. The archive contains 70 tracked package files. The extracted package smoke passed withoutnode_modules.Checklist