perf(ci): reuse dependency checks in release builds - #9399

Merged
t3dotgg merged 1 commit into
mainfrom
t3code/prioritize-windows-release-build
Sep 3, 2026
Merged

perf(ci): reuse dependency checks in release builds#9399
t3dotgg merged 1 commit into
mainfrom
t3code/prioritize-windows-release-build

Conversation

@t3dotgg

@t3dotggt3dotgg commented Sep 3, 2026

Copy link
Copy Markdown
Member

Windows release builds repeat pnpm's lockfile checks after preflight already ran them. A recent Windows run spent 49 seconds on these checks alongside package installation.

Pass preflight's small verification file to the desktop builds. pnpm checks the current lockfile and policy before reusing it. If the artifact is unavailable, installation runs the checks again. Build order, package caches, signing, and release quality checks stay unchanged.

Validation

  • Tested pnpm 11.10.0 with only the verification file copied to a new workspace. Missing files, changed lockfiles, and changed policies triggered verification again.
  • Targeted format checks passed. Actionlint found no new diagnostics against main.
  • Windows time savings are not measured. No release was triggered.

Implemented with GPT-5.6 Sol via Codex in T3 Code.


Note

Low Risk
CI-only workflow changes with graceful fallback when the artifact is absent; no application runtime or security logic changes.

Overview
Release desktop builds were redoing pnpm lockfile/policy checks after preflight had already run them (notably costly on Windows). This PR reuses only the small verification output instead of copying the full registry metadata cache.

Preflight now sets pnpm_config_cache_dir under the runner temp dir during install, then uploads lockfile-verified.jsonl as the release-dependency-verification artifact (continue-on-error: true). Matrix build jobs download that artifact into the same cache dir, set the same env on Install desktop dependencies and Build desktop artifact, so pnpm can skip redundant checks when the lockfile and policy still match. If the artifact is missing, install falls back to running verification again.

docs/internals/ci.md documents this handoff.

Reviewed by Cursor Bugbot for commit ee93f51. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Cache release preflight dependency checks for desktop build jobs

  • The release preflight job uploads a release-dependency-verification artifact containing lockfile-verified.jsonl.
  • Desktop dependency installation and artifact build jobs download this artifact into the pnpm metadata cache directory to reuse the verification data.
  • Updates docs/internals/ci.md to document the lockfile verification sharing mechanism.
  • Risk: Artifact upload and download steps are best-effort; failures do not fail the job and checks rerun normally.

Macroscope summarized ee93f51.

@github-actionsgithub-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels Sep 3, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ The exact PR base did not have a successful artifact. Baseline uses the latest successful main measurement shown below.

ProviderMetricMain baselineThis PRImpactPR ceiling
CodexTotal thread wire13.3 KiB13.6 KiB+289 B (+2.1%)15.1 KiB
CodexThread snapshot wire6.9 KiB7.0 KiB+128 B (+1.8%)7.3 KiB
CodexLive turn WebSocket wire6.4 KiB6.6 KiB+161 B (+2.5%)7.8 KiB
CodexLive turn WebSocket decoded55.6 KiB57.1 KiB+1.5 KiB (+2.7%)66.4 KiB
CodexLive turn messages10100 (0.0%)21
ClaudeTotal thread wire13.2 KiB13.6 KiB+341 B (+2.5%)15.1 KiB
ClaudeThread snapshot wire6.9 KiB7.0 KiB+144 B (+2.0%)7.3 KiB
ClaudeLive turn WebSocket wire6.3 KiB6.5 KiB+197 B (+3.0%)7.8 KiB
ClaudeLive turn WebSocket decoded56.3 KiB57.8 KiB+1.5 KiB (+2.7%)66.4 KiB
ClaudeLive turn messages89+1 (+12.5%)21

Baseline: 6ff537f · PR result: ee93f51 · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

  • Codex decoded thread snapshot: 113.8 KiB
  • Claude decoded thread snapshot: 114.5 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

@macroscopeapp

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at ee93f51

Macroscope's review found this PR approvable — This is a narrowly scoped CI performance change that shares pnpm verification metadata between release jobs without changing application behavior or release targets. Artifact failures safely fall back to the existing dependency checks, and the accompanying documentation has no runtime impact.

You can add or adjust custom eligibility rules. Learn more.

@t3dotgg
t3dotgg merged commit fff33f9 into mainSep 3, 2026
20 checks passed
@t3dotgg
t3dotgg deleted the t3code/prioritize-windows-release-build branch September 3, 2026 10:13
github-actionsBot added a commit to omarcresp/t3code-flake that referenced this pull request Sep 3, 2026
## What's Changed
* fix(mobile): show an error instead of an endless preview spinner by @t3dotgg in pingdotgg/t3code#9123
* fix(usage): price new models without waiting a day for the rate table by @t3dotgg in pingdotgg/t3code#9202
* fix(web): unlock the composer when preview capture fails by @t3dotgg in pingdotgg/t3code#9127
* fix(antigravity): refresh the model manifest so older Gemini models fold as legacy by @t3dotgg in pingdotgg/t3code#9397
* perf(ci): reuse dependency checks in release builds by @t3dotgg in pingdotgg/t3code#9399
**Full Changelog**: pingdotgg/t3code@v0.0.39-nightly.20260903.1268...v0.0.39-nightly.20260903.1270
Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.39-nightly.20260903.1270
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S10-29 changed lines (additions + deletions).vouch:trustedPR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@t3dotgg
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
Skip to content

perf(ci): reuse dependency checks in release builds - #9399

Merged
t3dotgg merged 1 commit into
mainfrom
t3code/prioritize-windows-release-build
Sep 3, 2026
Merged

perf(ci): reuse dependency checks in release builds#9399
t3dotgg merged 1 commit into
mainfrom
t3code/prioritize-windows-release-build

Conversation

@t3dotgg

@t3dotggt3dotgg commented Sep 3, 2026

Copy link
Copy Markdown
Member

Windows release builds repeat pnpm's lockfile checks after preflight already ran them. A recent Windows run spent 49 seconds on these checks alongside package installation.

Pass preflight's small verification file to the desktop builds. pnpm checks the current lockfile and policy before reusing it. If the artifact is unavailable, installation runs the checks again. Build order, package caches, signing, and release quality checks stay unchanged.

Validation

  • Tested pnpm 11.10.0 with only the verification file copied to a new workspace. Missing files, changed lockfiles, and changed policies triggered verification again.
  • Targeted format checks passed. Actionlint found no new diagnostics against main.
  • Windows time savings are not measured. No release was triggered.

Implemented with GPT-5.6 Sol via Codex in T3 Code.


Note

Low Risk
CI-only workflow changes with graceful fallback when the artifact is absent; no application runtime or security logic changes.

Overview
Release desktop builds were redoing pnpm lockfile/policy checks after preflight had already run them (notably costly on Windows). This PR reuses only the small verification output instead of copying the full registry metadata cache.

Preflight now sets pnpm_config_cache_dir under the runner temp dir during install, then uploads lockfile-verified.jsonl as the release-dependency-verification artifact (continue-on-error: true). Matrix build jobs download that artifact into the same cache dir, set the same env on Install desktop dependencies and Build desktop artifact, so pnpm can skip redundant checks when the lockfile and policy still match. If the artifact is missing, install falls back to running verification again.

docs/internals/ci.md documents this handoff.

Reviewed by Cursor Bugbot for commit ee93f51. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Cache release preflight dependency checks for desktop build jobs

  • The release preflight job uploads a release-dependency-verification artifact containing lockfile-verified.jsonl.
  • Desktop dependency installation and artifact build jobs download this artifact into the pnpm metadata cache directory to reuse the verification data.
  • Updates docs/internals/ci.md to document the lockfile verification sharing mechanism.
  • Risk: Artifact upload and download steps are best-effort; failures do not fail the job and checks rerun normally.

Macroscope summarized ee93f51.

@github-actionsgithub-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels Sep 3, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ The exact PR base did not have a successful artifact. Baseline uses the latest successful main measurement shown below.

ProviderMetricMain baselineThis PRImpactPR ceiling
CodexTotal thread wire13.3 KiB13.6 KiB+289 B (+2.1%)15.1 KiB
CodexThread snapshot wire6.9 KiB7.0 KiB+128 B (+1.8%)7.3 KiB
CodexLive turn WebSocket wire6.4 KiB6.6 KiB+161 B (+2.5%)7.8 KiB
CodexLive turn WebSocket decoded55.6 KiB57.1 KiB+1.5 KiB (+2.7%)66.4 KiB
CodexLive turn messages10100 (0.0%)21
ClaudeTotal thread wire13.2 KiB13.6 KiB+341 B (+2.5%)15.1 KiB
ClaudeThread snapshot wire6.9 KiB7.0 KiB+144 B (+2.0%)7.3 KiB
ClaudeLive turn WebSocket wire6.3 KiB6.5 KiB+197 B (+3.0%)7.8 KiB
ClaudeLive turn WebSocket decoded56.3 KiB57.8 KiB+1.5 KiB (+2.7%)66.4 KiB
ClaudeLive turn messages89+1 (+12.5%)21

Baseline: 6ff537f · PR result: ee93f51 · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

  • Codex decoded thread snapshot: 113.8 KiB
  • Claude decoded thread snapshot: 114.5 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

@macroscopeapp

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at ee93f51

Macroscope's review found this PR approvable — This is a narrowly scoped CI performance change that shares pnpm verification metadata between release jobs without changing application behavior or release targets. Artifact failures safely fall back to the existing dependency checks, and the accompanying documentation has no runtime impact.

You can add or adjust custom eligibility rules. Learn more.

@t3dotgg
t3dotgg merged commit fff33f9 into mainSep 3, 2026
20 checks passed
@t3dotgg
t3dotgg deleted the t3code/prioritize-windows-release-build branch September 3, 2026 10:13
github-actionsBot added a commit to omarcresp/t3code-flake that referenced this pull request Sep 3, 2026
## What's Changed
* fix(mobile): show an error instead of an endless preview spinner by @t3dotgg in pingdotgg/t3code#9123
* fix(usage): price new models without waiting a day for the rate table by @t3dotgg in pingdotgg/t3code#9202
* fix(web): unlock the composer when preview capture fails by @t3dotgg in pingdotgg/t3code#9127
* fix(antigravity): refresh the model manifest so older Gemini models fold as legacy by @t3dotgg in pingdotgg/t3code#9397
* perf(ci): reuse dependency checks in release builds by @t3dotgg in pingdotgg/t3code#9399
**Full Changelog**: pingdotgg/t3code@v0.0.39-nightly.20260903.1268...v0.0.39-nightly.20260903.1270
Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.39-nightly.20260903.1270
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S10-29 changed lines (additions + deletions).vouch:trustedPR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@t3dotgg
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix"); } } catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

perf(ci): reuse dependency checks in release builds - #9399

Merged
t3dotgg merged 1 commit into
mainfrom
t3code/prioritize-windows-release-build
Sep 3, 2026
Merged

perf(ci): reuse dependency checks in release builds#9399
t3dotgg merged 1 commit into
mainfrom
t3code/prioritize-windows-release-build

Conversation

@t3dotgg

@t3dotggt3dotgg commented Sep 3, 2026

Copy link
Copy Markdown
Member

Windows release builds repeat pnpm's lockfile checks after preflight already ran them. A recent Windows run spent 49 seconds on these checks alongside package installation.

Pass preflight's small verification file to the desktop builds. pnpm checks the current lockfile and policy before reusing it. If the artifact is unavailable, installation runs the checks again. Build order, package caches, signing, and release quality checks stay unchanged.

Validation

  • Tested pnpm 11.10.0 with only the verification file copied to a new workspace. Missing files, changed lockfiles, and changed policies triggered verification again.
  • Targeted format checks passed. Actionlint found no new diagnostics against main.
  • Windows time savings are not measured. No release was triggered.

Implemented with GPT-5.6 Sol via Codex in T3 Code.


Note

Low Risk
CI-only workflow changes with graceful fallback when the artifact is absent; no application runtime or security logic changes.

Overview
Release desktop builds were redoing pnpm lockfile/policy checks after preflight had already run them (notably costly on Windows). This PR reuses only the small verification output instead of copying the full registry metadata cache.

Preflight now sets pnpm_config_cache_dir under the runner temp dir during install, then uploads lockfile-verified.jsonl as the release-dependency-verification artifact (continue-on-error: true). Matrix build jobs download that artifact into the same cache dir, set the same env on Install desktop dependencies and Build desktop artifact, so pnpm can skip redundant checks when the lockfile and policy still match. If the artifact is missing, install falls back to running verification again.

docs/internals/ci.md documents this handoff.

Reviewed by Cursor Bugbot for commit ee93f51. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Cache release preflight dependency checks for desktop build jobs

  • The release preflight job uploads a release-dependency-verification artifact containing lockfile-verified.jsonl.
  • Desktop dependency installation and artifact build jobs download this artifact into the pnpm metadata cache directory to reuse the verification data.
  • Updates docs/internals/ci.md to document the lockfile verification sharing mechanism.
  • Risk: Artifact upload and download steps are best-effort; failures do not fail the job and checks rerun normally.

Macroscope summarized ee93f51.

@github-actionsgithub-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels Sep 3, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ The exact PR base did not have a successful artifact. Baseline uses the latest successful main measurement shown below.

ProviderMetricMain baselineThis PRImpactPR ceiling
CodexTotal thread wire13.3 KiB13.6 KiB+289 B (+2.1%)15.1 KiB
CodexThread snapshot wire6.9 KiB7.0 KiB+128 B (+1.8%)7.3 KiB
CodexLive turn WebSocket wire6.4 KiB6.6 KiB+161 B (+2.5%)7.8 KiB
CodexLive turn WebSocket decoded55.6 KiB57.1 KiB+1.5 KiB (+2.7%)66.4 KiB
CodexLive turn messages10100 (0.0%)21
ClaudeTotal thread wire13.2 KiB13.6 KiB+341 B (+2.5%)15.1 KiB
ClaudeThread snapshot wire6.9 KiB7.0 KiB+144 B (+2.0%)7.3 KiB
ClaudeLive turn WebSocket wire6.3 KiB6.5 KiB+197 B (+3.0%)7.8 KiB
ClaudeLive turn WebSocket decoded56.3 KiB57.8 KiB+1.5 KiB (+2.7%)66.4 KiB
ClaudeLive turn messages89+1 (+12.5%)21

Baseline: 6ff537f · PR result: ee93f51 · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

  • Codex decoded thread snapshot: 113.8 KiB
  • Claude decoded thread snapshot: 114.5 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

@macroscopeapp

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at ee93f51

Macroscope's review found this PR approvable — This is a narrowly scoped CI performance change that shares pnpm verification metadata between release jobs without changing application behavior or release targets. Artifact failures safely fall back to the existing dependency checks, and the accompanying documentation has no runtime impact.

You can add or adjust custom eligibility rules. Learn more.

@t3dotgg
t3dotgg merged commit fff33f9 into mainSep 3, 2026
20 checks passed
@t3dotgg
t3dotgg deleted the t3code/prioritize-windows-release-build branch September 3, 2026 10:13
github-actionsBot added a commit to omarcresp/t3code-flake that referenced this pull request Sep 3, 2026
## What's Changed
* fix(mobile): show an error instead of an endless preview spinner by @t3dotgg in pingdotgg/t3code#9123
* fix(usage): price new models without waiting a day for the rate table by @t3dotgg in pingdotgg/t3code#9202
* fix(web): unlock the composer when preview capture fails by @t3dotgg in pingdotgg/t3code#9127
* fix(antigravity): refresh the model manifest so older Gemini models fold as legacy by @t3dotgg in pingdotgg/t3code#9397
* perf(ci): reuse dependency checks in release builds by @t3dotgg in pingdotgg/t3code#9399
**Full Changelog**: pingdotgg/t3code@v0.0.39-nightly.20260903.1268...v0.0.39-nightly.20260903.1270
Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.39-nightly.20260903.1270
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S10-29 changed lines (additions + deletions).vouch:trustedPR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@t3dotgg
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms"); } } catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

perf(ci): reuse dependency checks in release builds - #9399

Merged
t3dotgg merged 1 commit into
mainfrom
t3code/prioritize-windows-release-build
Sep 3, 2026
Merged

perf(ci): reuse dependency checks in release builds#9399
t3dotgg merged 1 commit into
mainfrom
t3code/prioritize-windows-release-build

Conversation

@t3dotgg

@t3dotggt3dotgg commented Sep 3, 2026

Copy link
Copy Markdown
Member

Windows release builds repeat pnpm's lockfile checks after preflight already ran them. A recent Windows run spent 49 seconds on these checks alongside package installation.

Pass preflight's small verification file to the desktop builds. pnpm checks the current lockfile and policy before reusing it. If the artifact is unavailable, installation runs the checks again. Build order, package caches, signing, and release quality checks stay unchanged.

Validation

  • Tested pnpm 11.10.0 with only the verification file copied to a new workspace. Missing files, changed lockfiles, and changed policies triggered verification again.
  • Targeted format checks passed. Actionlint found no new diagnostics against main.
  • Windows time savings are not measured. No release was triggered.

Implemented with GPT-5.6 Sol via Codex in T3 Code.


Note

Low Risk
CI-only workflow changes with graceful fallback when the artifact is absent; no application runtime or security logic changes.

Overview
Release desktop builds were redoing pnpm lockfile/policy checks after preflight had already run them (notably costly on Windows). This PR reuses only the small verification output instead of copying the full registry metadata cache.

Preflight now sets pnpm_config_cache_dir under the runner temp dir during install, then uploads lockfile-verified.jsonl as the release-dependency-verification artifact (continue-on-error: true). Matrix build jobs download that artifact into the same cache dir, set the same env on Install desktop dependencies and Build desktop artifact, so pnpm can skip redundant checks when the lockfile and policy still match. If the artifact is missing, install falls back to running verification again.

docs/internals/ci.md documents this handoff.

Reviewed by Cursor Bugbot for commit ee93f51. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Cache release preflight dependency checks for desktop build jobs

  • The release preflight job uploads a release-dependency-verification artifact containing lockfile-verified.jsonl.
  • Desktop dependency installation and artifact build jobs download this artifact into the pnpm metadata cache directory to reuse the verification data.
  • Updates docs/internals/ci.md to document the lockfile verification sharing mechanism.
  • Risk: Artifact upload and download steps are best-effort; failures do not fail the job and checks rerun normally.

Macroscope summarized ee93f51.

@github-actionsgithub-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels Sep 3, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ The exact PR base did not have a successful artifact. Baseline uses the latest successful main measurement shown below.

ProviderMetricMain baselineThis PRImpactPR ceiling
CodexTotal thread wire13.3 KiB13.6 KiB+289 B (+2.1%)15.1 KiB
CodexThread snapshot wire6.9 KiB7.0 KiB+128 B (+1.8%)7.3 KiB
CodexLive turn WebSocket wire6.4 KiB6.6 KiB+161 B (+2.5%)7.8 KiB
CodexLive turn WebSocket decoded55.6 KiB57.1 KiB+1.5 KiB (+2.7%)66.4 KiB
CodexLive turn messages10100 (0.0%)21
ClaudeTotal thread wire13.2 KiB13.6 KiB+341 B (+2.5%)15.1 KiB
ClaudeThread snapshot wire6.9 KiB7.0 KiB+144 B (+2.0%)7.3 KiB
ClaudeLive turn WebSocket wire6.3 KiB6.5 KiB+197 B (+3.0%)7.8 KiB
ClaudeLive turn WebSocket decoded56.3 KiB57.8 KiB+1.5 KiB (+2.7%)66.4 KiB
ClaudeLive turn messages89+1 (+12.5%)21

Baseline: 6ff537f · PR result: ee93f51 · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

  • Codex decoded thread snapshot: 113.8 KiB
  • Claude decoded thread snapshot: 114.5 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

@macroscopeapp

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at ee93f51

Macroscope's review found this PR approvable — This is a narrowly scoped CI performance change that shares pnpm verification metadata between release jobs without changing application behavior or release targets. Artifact failures safely fall back to the existing dependency checks, and the accompanying documentation has no runtime impact.

You can add or adjust custom eligibility rules. Learn more.

@t3dotgg
t3dotgg merged commit fff33f9 into mainSep 3, 2026
20 checks passed
@t3dotgg
t3dotgg deleted the t3code/prioritize-windows-release-build branch September 3, 2026 10:13
github-actionsBot added a commit to omarcresp/t3code-flake that referenced this pull request Sep 3, 2026
## What's Changed
* fix(mobile): show an error instead of an endless preview spinner by @t3dotgg in pingdotgg/t3code#9123
* fix(usage): price new models without waiting a day for the rate table by @t3dotgg in pingdotgg/t3code#9202
* fix(web): unlock the composer when preview capture fails by @t3dotgg in pingdotgg/t3code#9127
* fix(antigravity): refresh the model manifest so older Gemini models fold as legacy by @t3dotgg in pingdotgg/t3code#9397
* perf(ci): reuse dependency checks in release builds by @t3dotgg in pingdotgg/t3code#9399
**Full Changelog**: pingdotgg/t3code@v0.0.39-nightly.20260903.1268...v0.0.39-nightly.20260903.1270
Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.39-nightly.20260903.1270
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S10-29 changed lines (additions + deletions).vouch:trustedPR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@t3dotgg
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links"); } } catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); } })(); (function(){ try { var __m = "youtube.com"; var __re = new RegExp('^' + "youtube\\.com" + '
Skip to content

perf(ci): reuse dependency checks in release builds - #9399

Merged
t3dotgg merged 1 commit into
mainfrom
t3code/prioritize-windows-release-build
Sep 3, 2026
Merged

perf(ci): reuse dependency checks in release builds#9399
t3dotgg merged 1 commit into
mainfrom
t3code/prioritize-windows-release-build

Conversation

@t3dotgg

@t3dotggt3dotgg commented Sep 3, 2026

Copy link
Copy Markdown
Member

Windows release builds repeat pnpm's lockfile checks after preflight already ran them. A recent Windows run spent 49 seconds on these checks alongside package installation.

Pass preflight's small verification file to the desktop builds. pnpm checks the current lockfile and policy before reusing it. If the artifact is unavailable, installation runs the checks again. Build order, package caches, signing, and release quality checks stay unchanged.

Validation

  • Tested pnpm 11.10.0 with only the verification file copied to a new workspace. Missing files, changed lockfiles, and changed policies triggered verification again.
  • Targeted format checks passed. Actionlint found no new diagnostics against main.
  • Windows time savings are not measured. No release was triggered.

Implemented with GPT-5.6 Sol via Codex in T3 Code.


Note

Low Risk
CI-only workflow changes with graceful fallback when the artifact is absent; no application runtime or security logic changes.

Overview
Release desktop builds were redoing pnpm lockfile/policy checks after preflight had already run them (notably costly on Windows). This PR reuses only the small verification output instead of copying the full registry metadata cache.

Preflight now sets pnpm_config_cache_dir under the runner temp dir during install, then uploads lockfile-verified.jsonl as the release-dependency-verification artifact (continue-on-error: true). Matrix build jobs download that artifact into the same cache dir, set the same env on Install desktop dependencies and Build desktop artifact, so pnpm can skip redundant checks when the lockfile and policy still match. If the artifact is missing, install falls back to running verification again.

docs/internals/ci.md documents this handoff.

Reviewed by Cursor Bugbot for commit ee93f51. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Cache release preflight dependency checks for desktop build jobs

  • The release preflight job uploads a release-dependency-verification artifact containing lockfile-verified.jsonl.
  • Desktop dependency installation and artifact build jobs download this artifact into the pnpm metadata cache directory to reuse the verification data.
  • Updates docs/internals/ci.md to document the lockfile verification sharing mechanism.
  • Risk: Artifact upload and download steps are best-effort; failures do not fail the job and checks rerun normally.

Macroscope summarized ee93f51.

@github-actionsgithub-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels Sep 3, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ The exact PR base did not have a successful artifact. Baseline uses the latest successful main measurement shown below.

ProviderMetricMain baselineThis PRImpactPR ceiling
CodexTotal thread wire13.3 KiB13.6 KiB+289 B (+2.1%)15.1 KiB
CodexThread snapshot wire6.9 KiB7.0 KiB+128 B (+1.8%)7.3 KiB
CodexLive turn WebSocket wire6.4 KiB6.6 KiB+161 B (+2.5%)7.8 KiB
CodexLive turn WebSocket decoded55.6 KiB57.1 KiB+1.5 KiB (+2.7%)66.4 KiB
CodexLive turn messages10100 (0.0%)21
ClaudeTotal thread wire13.2 KiB13.6 KiB+341 B (+2.5%)15.1 KiB
ClaudeThread snapshot wire6.9 KiB7.0 KiB+144 B (+2.0%)7.3 KiB
ClaudeLive turn WebSocket wire6.3 KiB6.5 KiB+197 B (+3.0%)7.8 KiB
ClaudeLive turn WebSocket decoded56.3 KiB57.8 KiB+1.5 KiB (+2.7%)66.4 KiB
ClaudeLive turn messages89+1 (+12.5%)21

Baseline: 6ff537f · PR result: ee93f51 · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

  • Codex decoded thread snapshot: 113.8 KiB
  • Claude decoded thread snapshot: 114.5 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

@macroscopeapp

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at ee93f51

Macroscope's review found this PR approvable — This is a narrowly scoped CI performance change that shares pnpm verification metadata between release jobs without changing application behavior or release targets. Artifact failures safely fall back to the existing dependency checks, and the accompanying documentation has no runtime impact.

You can add or adjust custom eligibility rules. Learn more.

@t3dotgg
t3dotgg merged commit fff33f9 into mainSep 3, 2026
20 checks passed
@t3dotgg
t3dotgg deleted the t3code/prioritize-windows-release-build branch September 3, 2026 10:13
github-actionsBot added a commit to omarcresp/t3code-flake that referenced this pull request Sep 3, 2026
## What's Changed
* fix(mobile): show an error instead of an endless preview spinner by @t3dotgg in pingdotgg/t3code#9123
* fix(usage): price new models without waiting a day for the rate table by @t3dotgg in pingdotgg/t3code#9202
* fix(web): unlock the composer when preview capture fails by @t3dotgg in pingdotgg/t3code#9127
* fix(antigravity): refresh the model manifest so older Gemini models fold as legacy by @t3dotgg in pingdotgg/t3code#9397
* perf(ci): reuse dependency checks in release builds by @t3dotgg in pingdotgg/t3code#9399
**Full Changelog**: pingdotgg/t3code@v0.0.39-nightly.20260903.1268...v0.0.39-nightly.20260903.1270
Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.39-nightly.20260903.1270
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S10-29 changed lines (additions + deletions).vouch:trustedPR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@t3dotgg
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default"); } } catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

perf(ci): reuse dependency checks in release builds - #9399

Merged
t3dotgg merged 1 commit into
mainfrom
t3code/prioritize-windows-release-build
Sep 3, 2026
Merged

perf(ci): reuse dependency checks in release builds#9399
t3dotgg merged 1 commit into
mainfrom
t3code/prioritize-windows-release-build

Conversation

@t3dotgg

@t3dotggt3dotgg commented Sep 3, 2026

Copy link
Copy Markdown
Member

Windows release builds repeat pnpm's lockfile checks after preflight already ran them. A recent Windows run spent 49 seconds on these checks alongside package installation.

Pass preflight's small verification file to the desktop builds. pnpm checks the current lockfile and policy before reusing it. If the artifact is unavailable, installation runs the checks again. Build order, package caches, signing, and release quality checks stay unchanged.

Validation

  • Tested pnpm 11.10.0 with only the verification file copied to a new workspace. Missing files, changed lockfiles, and changed policies triggered verification again.
  • Targeted format checks passed. Actionlint found no new diagnostics against main.
  • Windows time savings are not measured. No release was triggered.

Implemented with GPT-5.6 Sol via Codex in T3 Code.


Note

Low Risk
CI-only workflow changes with graceful fallback when the artifact is absent; no application runtime or security logic changes.

Overview
Release desktop builds were redoing pnpm lockfile/policy checks after preflight had already run them (notably costly on Windows). This PR reuses only the small verification output instead of copying the full registry metadata cache.

Preflight now sets pnpm_config_cache_dir under the runner temp dir during install, then uploads lockfile-verified.jsonl as the release-dependency-verification artifact (continue-on-error: true). Matrix build jobs download that artifact into the same cache dir, set the same env on Install desktop dependencies and Build desktop artifact, so pnpm can skip redundant checks when the lockfile and policy still match. If the artifact is missing, install falls back to running verification again.

docs/internals/ci.md documents this handoff.

Reviewed by Cursor Bugbot for commit ee93f51. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Cache release preflight dependency checks for desktop build jobs

  • The release preflight job uploads a release-dependency-verification artifact containing lockfile-verified.jsonl.
  • Desktop dependency installation and artifact build jobs download this artifact into the pnpm metadata cache directory to reuse the verification data.
  • Updates docs/internals/ci.md to document the lockfile verification sharing mechanism.
  • Risk: Artifact upload and download steps are best-effort; failures do not fail the job and checks rerun normally.

Macroscope summarized ee93f51.

@github-actionsgithub-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels Sep 3, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ The exact PR base did not have a successful artifact. Baseline uses the latest successful main measurement shown below.

ProviderMetricMain baselineThis PRImpactPR ceiling
CodexTotal thread wire13.3 KiB13.6 KiB+289 B (+2.1%)15.1 KiB
CodexThread snapshot wire6.9 KiB7.0 KiB+128 B (+1.8%)7.3 KiB
CodexLive turn WebSocket wire6.4 KiB6.6 KiB+161 B (+2.5%)7.8 KiB
CodexLive turn WebSocket decoded55.6 KiB57.1 KiB+1.5 KiB (+2.7%)66.4 KiB
CodexLive turn messages10100 (0.0%)21
ClaudeTotal thread wire13.2 KiB13.6 KiB+341 B (+2.5%)15.1 KiB
ClaudeThread snapshot wire6.9 KiB7.0 KiB+144 B (+2.0%)7.3 KiB
ClaudeLive turn WebSocket wire6.3 KiB6.5 KiB+197 B (+3.0%)7.8 KiB
ClaudeLive turn WebSocket decoded56.3 KiB57.8 KiB+1.5 KiB (+2.7%)66.4 KiB
ClaudeLive turn messages89+1 (+12.5%)21

Baseline: 6ff537f · PR result: ee93f51 · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

  • Codex decoded thread snapshot: 113.8 KiB
  • Claude decoded thread snapshot: 114.5 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

@macroscopeapp

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at ee93f51

Macroscope's review found this PR approvable — This is a narrowly scoped CI performance change that shares pnpm verification metadata between release jobs without changing application behavior or release targets. Artifact failures safely fall back to the existing dependency checks, and the accompanying documentation has no runtime impact.

You can add or adjust custom eligibility rules. Learn more.

@t3dotgg
t3dotgg merged commit fff33f9 into mainSep 3, 2026
20 checks passed
@t3dotgg
t3dotgg deleted the t3code/prioritize-windows-release-build branch September 3, 2026 10:13
github-actionsBot added a commit to omarcresp/t3code-flake that referenced this pull request Sep 3, 2026
## What's Changed
* fix(mobile): show an error instead of an endless preview spinner by @t3dotgg in pingdotgg/t3code#9123
* fix(usage): price new models without waiting a day for the rate table by @t3dotgg in pingdotgg/t3code#9202
* fix(web): unlock the composer when preview capture fails by @t3dotgg in pingdotgg/t3code#9127
* fix(antigravity): refresh the model manifest so older Gemini models fold as legacy by @t3dotgg in pingdotgg/t3code#9397
* perf(ci): reuse dependency checks in release builds by @t3dotgg in pingdotgg/t3code#9399
**Full Changelog**: pingdotgg/t3code@v0.0.39-nightly.20260903.1268...v0.0.39-nightly.20260903.1270
Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.39-nightly.20260903.1270
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S10-29 changed lines (additions + deletions).vouch:trustedPR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@t3dotgg
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers"); } } catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

perf(ci): reuse dependency checks in release builds - #9399

Merged
t3dotgg merged 1 commit into
mainfrom
t3code/prioritize-windows-release-build
Sep 3, 2026
Merged

perf(ci): reuse dependency checks in release builds#9399
t3dotgg merged 1 commit into
mainfrom
t3code/prioritize-windows-release-build

Conversation

@t3dotgg

@t3dotggt3dotgg commented Sep 3, 2026

Copy link
Copy Markdown
Member

Windows release builds repeat pnpm's lockfile checks after preflight already ran them. A recent Windows run spent 49 seconds on these checks alongside package installation.

Pass preflight's small verification file to the desktop builds. pnpm checks the current lockfile and policy before reusing it. If the artifact is unavailable, installation runs the checks again. Build order, package caches, signing, and release quality checks stay unchanged.

Validation

  • Tested pnpm 11.10.0 with only the verification file copied to a new workspace. Missing files, changed lockfiles, and changed policies triggered verification again.
  • Targeted format checks passed. Actionlint found no new diagnostics against main.
  • Windows time savings are not measured. No release was triggered.

Implemented with GPT-5.6 Sol via Codex in T3 Code.


Note

Low Risk
CI-only workflow changes with graceful fallback when the artifact is absent; no application runtime or security logic changes.

Overview
Release desktop builds were redoing pnpm lockfile/policy checks after preflight had already run them (notably costly on Windows). This PR reuses only the small verification output instead of copying the full registry metadata cache.

Preflight now sets pnpm_config_cache_dir under the runner temp dir during install, then uploads lockfile-verified.jsonl as the release-dependency-verification artifact (continue-on-error: true). Matrix build jobs download that artifact into the same cache dir, set the same env on Install desktop dependencies and Build desktop artifact, so pnpm can skip redundant checks when the lockfile and policy still match. If the artifact is missing, install falls back to running verification again.

docs/internals/ci.md documents this handoff.

Reviewed by Cursor Bugbot for commit ee93f51. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Cache release preflight dependency checks for desktop build jobs

  • The release preflight job uploads a release-dependency-verification artifact containing lockfile-verified.jsonl.
  • Desktop dependency installation and artifact build jobs download this artifact into the pnpm metadata cache directory to reuse the verification data.
  • Updates docs/internals/ci.md to document the lockfile verification sharing mechanism.
  • Risk: Artifact upload and download steps are best-effort; failures do not fail the job and checks rerun normally.

Macroscope summarized ee93f51.

@github-actionsgithub-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels Sep 3, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ The exact PR base did not have a successful artifact. Baseline uses the latest successful main measurement shown below.

ProviderMetricMain baselineThis PRImpactPR ceiling
CodexTotal thread wire13.3 KiB13.6 KiB+289 B (+2.1%)15.1 KiB
CodexThread snapshot wire6.9 KiB7.0 KiB+128 B (+1.8%)7.3 KiB
CodexLive turn WebSocket wire6.4 KiB6.6 KiB+161 B (+2.5%)7.8 KiB
CodexLive turn WebSocket decoded55.6 KiB57.1 KiB+1.5 KiB (+2.7%)66.4 KiB
CodexLive turn messages10100 (0.0%)21
ClaudeTotal thread wire13.2 KiB13.6 KiB+341 B (+2.5%)15.1 KiB
ClaudeThread snapshot wire6.9 KiB7.0 KiB+144 B (+2.0%)7.3 KiB
ClaudeLive turn WebSocket wire6.3 KiB6.5 KiB+197 B (+3.0%)7.8 KiB
ClaudeLive turn WebSocket decoded56.3 KiB57.8 KiB+1.5 KiB (+2.7%)66.4 KiB
ClaudeLive turn messages89+1 (+12.5%)21

Baseline: 6ff537f · PR result: ee93f51 · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

  • Codex decoded thread snapshot: 113.8 KiB
  • Claude decoded thread snapshot: 114.5 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

@macroscopeapp

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at ee93f51

Macroscope's review found this PR approvable — This is a narrowly scoped CI performance change that shares pnpm verification metadata between release jobs without changing application behavior or release targets. Artifact failures safely fall back to the existing dependency checks, and the accompanying documentation has no runtime impact.

You can add or adjust custom eligibility rules. Learn more.

@t3dotgg
t3dotgg merged commit fff33f9 into mainSep 3, 2026
20 checks passed
@t3dotgg
t3dotgg deleted the t3code/prioritize-windows-release-build branch September 3, 2026 10:13
github-actionsBot added a commit to omarcresp/t3code-flake that referenced this pull request Sep 3, 2026
## What's Changed
* fix(mobile): show an error instead of an endless preview spinner by @t3dotgg in pingdotgg/t3code#9123
* fix(usage): price new models without waiting a day for the rate table by @t3dotgg in pingdotgg/t3code#9202
* fix(web): unlock the composer when preview capture fails by @t3dotgg in pingdotgg/t3code#9127
* fix(antigravity): refresh the model manifest so older Gemini models fold as legacy by @t3dotgg in pingdotgg/t3code#9397
* perf(ci): reuse dependency checks in release builds by @t3dotgg in pingdotgg/t3code#9399
**Full Changelog**: pingdotgg/t3code@v0.0.39-nightly.20260903.1268...v0.0.39-nightly.20260903.1270
Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.39-nightly.20260903.1270
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S10-29 changed lines (additions + deletions).vouch:trustedPR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@t3dotgg
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode"); } } catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); } })(); })();
Skip to content

perf(ci): reuse dependency checks in release builds - #9399

Merged
t3dotgg merged 1 commit into
mainfrom
t3code/prioritize-windows-release-build
Sep 3, 2026
Merged

perf(ci): reuse dependency checks in release builds#9399
t3dotgg merged 1 commit into
mainfrom
t3code/prioritize-windows-release-build

Conversation

@t3dotgg

@t3dotggt3dotgg commented Sep 3, 2026

Copy link
Copy Markdown
Member

Windows release builds repeat pnpm's lockfile checks after preflight already ran them. A recent Windows run spent 49 seconds on these checks alongside package installation.

Pass preflight's small verification file to the desktop builds. pnpm checks the current lockfile and policy before reusing it. If the artifact is unavailable, installation runs the checks again. Build order, package caches, signing, and release quality checks stay unchanged.

Validation

  • Tested pnpm 11.10.0 with only the verification file copied to a new workspace. Missing files, changed lockfiles, and changed policies triggered verification again.
  • Targeted format checks passed. Actionlint found no new diagnostics against main.
  • Windows time savings are not measured. No release was triggered.

Implemented with GPT-5.6 Sol via Codex in T3 Code.


Note

Low Risk
CI-only workflow changes with graceful fallback when the artifact is absent; no application runtime or security logic changes.

Overview
Release desktop builds were redoing pnpm lockfile/policy checks after preflight had already run them (notably costly on Windows). This PR reuses only the small verification output instead of copying the full registry metadata cache.

Preflight now sets pnpm_config_cache_dir under the runner temp dir during install, then uploads lockfile-verified.jsonl as the release-dependency-verification artifact (continue-on-error: true). Matrix build jobs download that artifact into the same cache dir, set the same env on Install desktop dependencies and Build desktop artifact, so pnpm can skip redundant checks when the lockfile and policy still match. If the artifact is missing, install falls back to running verification again.

docs/internals/ci.md documents this handoff.

Reviewed by Cursor Bugbot for commit ee93f51. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Cache release preflight dependency checks for desktop build jobs

  • The release preflight job uploads a release-dependency-verification artifact containing lockfile-verified.jsonl.
  • Desktop dependency installation and artifact build jobs download this artifact into the pnpm metadata cache directory to reuse the verification data.
  • Updates docs/internals/ci.md to document the lockfile verification sharing mechanism.
  • Risk: Artifact upload and download steps are best-effort; failures do not fail the job and checks rerun normally.

Macroscope summarized ee93f51.

@github-actionsgithub-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels Sep 3, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ The exact PR base did not have a successful artifact. Baseline uses the latest successful main measurement shown below.

ProviderMetricMain baselineThis PRImpactPR ceiling
CodexTotal thread wire13.3 KiB13.6 KiB+289 B (+2.1%)15.1 KiB
CodexThread snapshot wire6.9 KiB7.0 KiB+128 B (+1.8%)7.3 KiB
CodexLive turn WebSocket wire6.4 KiB6.6 KiB+161 B (+2.5%)7.8 KiB
CodexLive turn WebSocket decoded55.6 KiB57.1 KiB+1.5 KiB (+2.7%)66.4 KiB
CodexLive turn messages10100 (0.0%)21
ClaudeTotal thread wire13.2 KiB13.6 KiB+341 B (+2.5%)15.1 KiB
ClaudeThread snapshot wire6.9 KiB7.0 KiB+144 B (+2.0%)7.3 KiB
ClaudeLive turn WebSocket wire6.3 KiB6.5 KiB+197 B (+3.0%)7.8 KiB
ClaudeLive turn WebSocket decoded56.3 KiB57.8 KiB+1.5 KiB (+2.7%)66.4 KiB
ClaudeLive turn messages89+1 (+12.5%)21

Baseline: 6ff537f · PR result: ee93f51 · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

  • Codex decoded thread snapshot: 113.8 KiB
  • Claude decoded thread snapshot: 114.5 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

@macroscopeapp

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at ee93f51

Macroscope's review found this PR approvable — This is a narrowly scoped CI performance change that shares pnpm verification metadata between release jobs without changing application behavior or release targets. Artifact failures safely fall back to the existing dependency checks, and the accompanying documentation has no runtime impact.

You can add or adjust custom eligibility rules. Learn more.

@t3dotgg
t3dotgg merged commit fff33f9 into mainSep 3, 2026
20 checks passed
@t3dotgg
t3dotgg deleted the t3code/prioritize-windows-release-build branch September 3, 2026 10:13
github-actionsBot added a commit to omarcresp/t3code-flake that referenced this pull request Sep 3, 2026
## What's Changed
* fix(mobile): show an error instead of an endless preview spinner by @t3dotgg in pingdotgg/t3code#9123
* fix(usage): price new models without waiting a day for the rate table by @t3dotgg in pingdotgg/t3code#9202
* fix(web): unlock the composer when preview capture fails by @t3dotgg in pingdotgg/t3code#9127
* fix(antigravity): refresh the model manifest so older Gemini models fold as legacy by @t3dotgg in pingdotgg/t3code#9397
* perf(ci): reuse dependency checks in release builds by @t3dotgg in pingdotgg/t3code#9399
**Full Changelog**: pingdotgg/t3code@v0.0.39-nightly.20260903.1268...v0.0.39-nightly.20260903.1270
Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.39-nightly.20260903.1270
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S10-29 changed lines (additions + deletions).vouch:trustedPR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@t3dotgg