Add Ubuntu deb release artifact - #2474

Closed
Abhishek-Gawade-programmer wants to merge 2 commits into
pingdotgg:mainfrom
Abhishek-Gawade-programmer:codex/add-ubuntu-deb-release
Closed

Add Ubuntu deb release artifact#2474
Abhishek-Gawade-programmer wants to merge 2 commits into
pingdotgg:mainfrom
Abhishek-Gawade-programmer:codex/add-ubuntu-deb-release

Conversation

@Abhishek-Gawade-programmer

@Abhishek-Gawade-programmerAbhishek-Gawade-programmer commented May 2, 2026

Copy link
Copy Markdown

What Changed

  • added a dist:desktop:deb script for building a Linux .deb artifact
  • updated the release workflow to build and publish an Ubuntu .deb alongside the existing Linux AppImage artifact
  • updated the release docs to mention the .deb artifact and clarify that Linux auto-update remains AppImage-only

Why

Ubuntu users currently only get the AppImage artifact from the desktop release pipeline.

This keeps the existing AppImage path intact for Linux auto-update behavior, while also publishing a native .deb package for
direct Ubuntu installation. That keeps the change small and avoids changing the current updater assumptions.

Checklist

  • This PR is small and focused
  • I explained what changed and why
  • I included before/after screenshots for any UI changes
  • I included a video for animation/interaction changes

Note

Medium Risk
Modifies the release CI matrix and asset publishing steps; mistakes could break or delay desktop releases, but changes are additive and scoped to packaging/publishing.

Overview
Adds an Ubuntu x64 .deb to the desktop release pipeline alongside the existing Linux AppImage.

The GitHub Actions release.yml build matrix now runs a Linux deb target and the workflow copies/uploads *.deb assets into the GitHub Release. A new dist:desktop:deb script is added for local builds, and release docs are updated to note the new artifact and that Linux auto-update remains AppImage-based.

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

Note

Add Ubuntu x64 .deb artifact to the release build and publish pipeline

  • Adds a Linux Ubuntu x64 .deb target to the build matrix in release.yml, and includes *.deb files in both the file-copy and GitHub release upload steps.
  • Adds a dist:desktop:deb npm script in package.json for local .deb builds.
  • Updates docs/release.md to note that five artifacts are now built, and clarifies that the .deb is for direct installation while AppImage targets auto-update.

Macroscope summarized 009af84.

@coderabbitai

coderabbitaiBot commented May 2, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 293a6421-e8c1-48fd-bf88-fca2993beadc

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actionsgithub-actionsBot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels May 2, 2026
runner: blacksmith-32vcpu-ubuntu-2404
platform: linux
target: deb
arch: x64

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Duplicate artifact name causes release workflow failure

High Severity

The new Ubuntu .deb x64 matrix entry has platform: linux and arch: x64, identical to the existing Linux x64 (AppImage) entry. The artifact upload step on line 357 derives its name from desktop-${{ matrix.platform }}-${{ matrix.arch }}, so both jobs produce desktop-linux-x64. With actions/upload-artifact@v7, the second job to finish will fail because an artifact with that name already exists and no overwrite: true is set. The artifact name needs to incorporate the target to differentiate the two Linux builds.

Additional Locations (1)
Fix in CursorFix in Web

Reviewed by Cursor Bugbot for commit fbd1741. Configure here.

@macroscopeapp

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Needs human review

CI/CD changes to add Ubuntu .deb release artifact. An unresolved high-severity review comment identifies that the new matrix entry would create duplicate artifact names with the existing Linux AppImage build, causing workflow failures. This bug should be addressed before merging.

You can customize Macroscope's approvability policy. Learn more.

@cursorcursorBot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

There are 2 total unresolved issues (including 1 from previous review).

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 009af84. Configure here.

runner: blacksmith-32vcpu-ubuntu-2404
platform: linux
target: deb
arch: x64

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Linux updater metadata collision between AppImage and deb builds

Medium Severity

Both the AppImage and deb builds will generate identically-named auto-update metadata files (latest-linux.yml or nightly-linux.yml) because resolveGitHubPublishConfig injects a publish config for all targets. When artifacts are merged via download-artifact with merge-multiple: true, one metadata file silently overwrites the other. If the deb metadata wins, AppImage auto-update users get pointed to a .deb file, breaking auto-updates. The macOS builds handle a similar multi-arch collision via manifest renaming (lines 335–340), but no equivalent handling exists for the two Linux targets.

Additional Locations (1)
Fix in CursorFix in Web

Reviewed by Cursor Bugbot for commit 009af84. Configure here.

@Samueru-sama

Copy link
Copy Markdown

Would be great if this got merged.

I want to make a better appimage release in pkgforge-dev/Anylinux-AppImages#582 since the current appimage being released only works on new glibc systems and still has a libfuse2 dependency...

For that to happen I need the bare binary, the .deb is good enough for this.

@xlc-dev

Copy link
Copy Markdown

+1 for getting this merged. would be awesome

@juliusmarminge

Copy link
Copy Markdown
Member

Closing as part of an open-PR triage — #1655 is being kept as the active candidate for deb packaging (full pipeline). Thanks!

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:unvouchedPR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@Abhishek-Gawade-programmer@Samueru-sama@xlc-dev@juliusmarminge
, '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

Add Ubuntu deb release artifact - #2474

Closed
Abhishek-Gawade-programmer wants to merge 2 commits into
pingdotgg:mainfrom
Abhishek-Gawade-programmer:codex/add-ubuntu-deb-release
Closed

Add Ubuntu deb release artifact#2474
Abhishek-Gawade-programmer wants to merge 2 commits into
pingdotgg:mainfrom
Abhishek-Gawade-programmer:codex/add-ubuntu-deb-release

Conversation

@Abhishek-Gawade-programmer

@Abhishek-Gawade-programmerAbhishek-Gawade-programmer commented May 2, 2026

Copy link
Copy Markdown

What Changed

  • added a dist:desktop:deb script for building a Linux .deb artifact
  • updated the release workflow to build and publish an Ubuntu .deb alongside the existing Linux AppImage artifact
  • updated the release docs to mention the .deb artifact and clarify that Linux auto-update remains AppImage-only

Why

Ubuntu users currently only get the AppImage artifact from the desktop release pipeline.

This keeps the existing AppImage path intact for Linux auto-update behavior, while also publishing a native .deb package for
direct Ubuntu installation. That keeps the change small and avoids changing the current updater assumptions.

Checklist

  • This PR is small and focused
  • I explained what changed and why
  • I included before/after screenshots for any UI changes
  • I included a video for animation/interaction changes

Note

Medium Risk
Modifies the release CI matrix and asset publishing steps; mistakes could break or delay desktop releases, but changes are additive and scoped to packaging/publishing.

Overview
Adds an Ubuntu x64 .deb to the desktop release pipeline alongside the existing Linux AppImage.

The GitHub Actions release.yml build matrix now runs a Linux deb target and the workflow copies/uploads *.deb assets into the GitHub Release. A new dist:desktop:deb script is added for local builds, and release docs are updated to note the new artifact and that Linux auto-update remains AppImage-based.

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

Note

Add Ubuntu x64 .deb artifact to the release build and publish pipeline

  • Adds a Linux Ubuntu x64 .deb target to the build matrix in release.yml, and includes *.deb files in both the file-copy and GitHub release upload steps.
  • Adds a dist:desktop:deb npm script in package.json for local .deb builds.
  • Updates docs/release.md to note that five artifacts are now built, and clarifies that the .deb is for direct installation while AppImage targets auto-update.

Macroscope summarized 009af84.

@coderabbitai

coderabbitaiBot commented May 2, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 293a6421-e8c1-48fd-bf88-fca2993beadc

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actionsgithub-actionsBot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels May 2, 2026
runner: blacksmith-32vcpu-ubuntu-2404
platform: linux
target: deb
arch: x64

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Duplicate artifact name causes release workflow failure

High Severity

The new Ubuntu .deb x64 matrix entry has platform: linux and arch: x64, identical to the existing Linux x64 (AppImage) entry. The artifact upload step on line 357 derives its name from desktop-${{ matrix.platform }}-${{ matrix.arch }}, so both jobs produce desktop-linux-x64. With actions/upload-artifact@v7, the second job to finish will fail because an artifact with that name already exists and no overwrite: true is set. The artifact name needs to incorporate the target to differentiate the two Linux builds.

Additional Locations (1)
Fix in CursorFix in Web

Reviewed by Cursor Bugbot for commit fbd1741. Configure here.

@macroscopeapp

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Needs human review

CI/CD changes to add Ubuntu .deb release artifact. An unresolved high-severity review comment identifies that the new matrix entry would create duplicate artifact names with the existing Linux AppImage build, causing workflow failures. This bug should be addressed before merging.

You can customize Macroscope's approvability policy. Learn more.

@cursorcursorBot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

There are 2 total unresolved issues (including 1 from previous review).

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 009af84. Configure here.

runner: blacksmith-32vcpu-ubuntu-2404
platform: linux
target: deb
arch: x64

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Linux updater metadata collision between AppImage and deb builds

Medium Severity

Both the AppImage and deb builds will generate identically-named auto-update metadata files (latest-linux.yml or nightly-linux.yml) because resolveGitHubPublishConfig injects a publish config for all targets. When artifacts are merged via download-artifact with merge-multiple: true, one metadata file silently overwrites the other. If the deb metadata wins, AppImage auto-update users get pointed to a .deb file, breaking auto-updates. The macOS builds handle a similar multi-arch collision via manifest renaming (lines 335–340), but no equivalent handling exists for the two Linux targets.

Additional Locations (1)
Fix in CursorFix in Web

Reviewed by Cursor Bugbot for commit 009af84. Configure here.

@Samueru-sama

Copy link
Copy Markdown

Would be great if this got merged.

I want to make a better appimage release in pkgforge-dev/Anylinux-AppImages#582 since the current appimage being released only works on new glibc systems and still has a libfuse2 dependency...

For that to happen I need the bare binary, the .deb is good enough for this.

@xlc-dev

Copy link
Copy Markdown

+1 for getting this merged. would be awesome

@juliusmarminge

Copy link
Copy Markdown
Member

Closing as part of an open-PR triage — #1655 is being kept as the active candidate for deb packaging (full pipeline). Thanks!

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:unvouchedPR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@Abhishek-Gawade-programmer@Samueru-sama@xlc-dev@juliusmarminge
, '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

Add Ubuntu deb release artifact - #2474

Closed
Abhishek-Gawade-programmer wants to merge 2 commits into
pingdotgg:mainfrom
Abhishek-Gawade-programmer:codex/add-ubuntu-deb-release
Closed

Add Ubuntu deb release artifact#2474
Abhishek-Gawade-programmer wants to merge 2 commits into
pingdotgg:mainfrom
Abhishek-Gawade-programmer:codex/add-ubuntu-deb-release

Conversation

@Abhishek-Gawade-programmer

@Abhishek-Gawade-programmerAbhishek-Gawade-programmer commented May 2, 2026

Copy link
Copy Markdown

What Changed

  • added a dist:desktop:deb script for building a Linux .deb artifact
  • updated the release workflow to build and publish an Ubuntu .deb alongside the existing Linux AppImage artifact
  • updated the release docs to mention the .deb artifact and clarify that Linux auto-update remains AppImage-only

Why

Ubuntu users currently only get the AppImage artifact from the desktop release pipeline.

This keeps the existing AppImage path intact for Linux auto-update behavior, while also publishing a native .deb package for
direct Ubuntu installation. That keeps the change small and avoids changing the current updater assumptions.

Checklist

  • This PR is small and focused
  • I explained what changed and why
  • I included before/after screenshots for any UI changes
  • I included a video for animation/interaction changes

Note

Medium Risk
Modifies the release CI matrix and asset publishing steps; mistakes could break or delay desktop releases, but changes are additive and scoped to packaging/publishing.

Overview
Adds an Ubuntu x64 .deb to the desktop release pipeline alongside the existing Linux AppImage.

The GitHub Actions release.yml build matrix now runs a Linux deb target and the workflow copies/uploads *.deb assets into the GitHub Release. A new dist:desktop:deb script is added for local builds, and release docs are updated to note the new artifact and that Linux auto-update remains AppImage-based.

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

Note

Add Ubuntu x64 .deb artifact to the release build and publish pipeline

  • Adds a Linux Ubuntu x64 .deb target to the build matrix in release.yml, and includes *.deb files in both the file-copy and GitHub release upload steps.
  • Adds a dist:desktop:deb npm script in package.json for local .deb builds.
  • Updates docs/release.md to note that five artifacts are now built, and clarifies that the .deb is for direct installation while AppImage targets auto-update.

Macroscope summarized 009af84.

@coderabbitai

coderabbitaiBot commented May 2, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 293a6421-e8c1-48fd-bf88-fca2993beadc

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actionsgithub-actionsBot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels May 2, 2026
runner: blacksmith-32vcpu-ubuntu-2404
platform: linux
target: deb
arch: x64

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Duplicate artifact name causes release workflow failure

High Severity

The new Ubuntu .deb x64 matrix entry has platform: linux and arch: x64, identical to the existing Linux x64 (AppImage) entry. The artifact upload step on line 357 derives its name from desktop-${{ matrix.platform }}-${{ matrix.arch }}, so both jobs produce desktop-linux-x64. With actions/upload-artifact@v7, the second job to finish will fail because an artifact with that name already exists and no overwrite: true is set. The artifact name needs to incorporate the target to differentiate the two Linux builds.

Additional Locations (1)
Fix in CursorFix in Web

Reviewed by Cursor Bugbot for commit fbd1741. Configure here.

@macroscopeapp

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Needs human review

CI/CD changes to add Ubuntu .deb release artifact. An unresolved high-severity review comment identifies that the new matrix entry would create duplicate artifact names with the existing Linux AppImage build, causing workflow failures. This bug should be addressed before merging.

You can customize Macroscope's approvability policy. Learn more.

@cursorcursorBot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

There are 2 total unresolved issues (including 1 from previous review).

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 009af84. Configure here.

runner: blacksmith-32vcpu-ubuntu-2404
platform: linux
target: deb
arch: x64

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Linux updater metadata collision between AppImage and deb builds

Medium Severity

Both the AppImage and deb builds will generate identically-named auto-update metadata files (latest-linux.yml or nightly-linux.yml) because resolveGitHubPublishConfig injects a publish config for all targets. When artifacts are merged via download-artifact with merge-multiple: true, one metadata file silently overwrites the other. If the deb metadata wins, AppImage auto-update users get pointed to a .deb file, breaking auto-updates. The macOS builds handle a similar multi-arch collision via manifest renaming (lines 335–340), but no equivalent handling exists for the two Linux targets.

Additional Locations (1)
Fix in CursorFix in Web

Reviewed by Cursor Bugbot for commit 009af84. Configure here.

@Samueru-sama

Copy link
Copy Markdown

Would be great if this got merged.

I want to make a better appimage release in pkgforge-dev/Anylinux-AppImages#582 since the current appimage being released only works on new glibc systems and still has a libfuse2 dependency...

For that to happen I need the bare binary, the .deb is good enough for this.

@xlc-dev

Copy link
Copy Markdown

+1 for getting this merged. would be awesome

@juliusmarminge

Copy link
Copy Markdown
Member

Closing as part of an open-PR triage — #1655 is being kept as the active candidate for deb packaging (full pipeline). Thanks!

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:unvouchedPR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@Abhishek-Gawade-programmer@Samueru-sama@xlc-dev@juliusmarminge
, '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

Add Ubuntu deb release artifact - #2474

Closed
Abhishek-Gawade-programmer wants to merge 2 commits into
pingdotgg:mainfrom
Abhishek-Gawade-programmer:codex/add-ubuntu-deb-release
Closed

Add Ubuntu deb release artifact#2474
Abhishek-Gawade-programmer wants to merge 2 commits into
pingdotgg:mainfrom
Abhishek-Gawade-programmer:codex/add-ubuntu-deb-release

Conversation

@Abhishek-Gawade-programmer

@Abhishek-Gawade-programmerAbhishek-Gawade-programmer commented May 2, 2026

Copy link
Copy Markdown

What Changed

  • added a dist:desktop:deb script for building a Linux .deb artifact
  • updated the release workflow to build and publish an Ubuntu .deb alongside the existing Linux AppImage artifact
  • updated the release docs to mention the .deb artifact and clarify that Linux auto-update remains AppImage-only

Why

Ubuntu users currently only get the AppImage artifact from the desktop release pipeline.

This keeps the existing AppImage path intact for Linux auto-update behavior, while also publishing a native .deb package for
direct Ubuntu installation. That keeps the change small and avoids changing the current updater assumptions.

Checklist

  • This PR is small and focused
  • I explained what changed and why
  • I included before/after screenshots for any UI changes
  • I included a video for animation/interaction changes

Note

Medium Risk
Modifies the release CI matrix and asset publishing steps; mistakes could break or delay desktop releases, but changes are additive and scoped to packaging/publishing.

Overview
Adds an Ubuntu x64 .deb to the desktop release pipeline alongside the existing Linux AppImage.

The GitHub Actions release.yml build matrix now runs a Linux deb target and the workflow copies/uploads *.deb assets into the GitHub Release. A new dist:desktop:deb script is added for local builds, and release docs are updated to note the new artifact and that Linux auto-update remains AppImage-based.

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

Note

Add Ubuntu x64 .deb artifact to the release build and publish pipeline

  • Adds a Linux Ubuntu x64 .deb target to the build matrix in release.yml, and includes *.deb files in both the file-copy and GitHub release upload steps.
  • Adds a dist:desktop:deb npm script in package.json for local .deb builds.
  • Updates docs/release.md to note that five artifacts are now built, and clarifies that the .deb is for direct installation while AppImage targets auto-update.

Macroscope summarized 009af84.

@coderabbitai

coderabbitaiBot commented May 2, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 293a6421-e8c1-48fd-bf88-fca2993beadc

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actionsgithub-actionsBot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels May 2, 2026
runner: blacksmith-32vcpu-ubuntu-2404
platform: linux
target: deb
arch: x64

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Duplicate artifact name causes release workflow failure

High Severity

The new Ubuntu .deb x64 matrix entry has platform: linux and arch: x64, identical to the existing Linux x64 (AppImage) entry. The artifact upload step on line 357 derives its name from desktop-${{ matrix.platform }}-${{ matrix.arch }}, so both jobs produce desktop-linux-x64. With actions/upload-artifact@v7, the second job to finish will fail because an artifact with that name already exists and no overwrite: true is set. The artifact name needs to incorporate the target to differentiate the two Linux builds.

Additional Locations (1)
Fix in CursorFix in Web

Reviewed by Cursor Bugbot for commit fbd1741. Configure here.

@macroscopeapp

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Needs human review

CI/CD changes to add Ubuntu .deb release artifact. An unresolved high-severity review comment identifies that the new matrix entry would create duplicate artifact names with the existing Linux AppImage build, causing workflow failures. This bug should be addressed before merging.

You can customize Macroscope's approvability policy. Learn more.

@cursorcursorBot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

There are 2 total unresolved issues (including 1 from previous review).

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 009af84. Configure here.

runner: blacksmith-32vcpu-ubuntu-2404
platform: linux
target: deb
arch: x64

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Linux updater metadata collision between AppImage and deb builds

Medium Severity

Both the AppImage and deb builds will generate identically-named auto-update metadata files (latest-linux.yml or nightly-linux.yml) because resolveGitHubPublishConfig injects a publish config for all targets. When artifacts are merged via download-artifact with merge-multiple: true, one metadata file silently overwrites the other. If the deb metadata wins, AppImage auto-update users get pointed to a .deb file, breaking auto-updates. The macOS builds handle a similar multi-arch collision via manifest renaming (lines 335–340), but no equivalent handling exists for the two Linux targets.

Additional Locations (1)
Fix in CursorFix in Web

Reviewed by Cursor Bugbot for commit 009af84. Configure here.

@Samueru-sama

Copy link
Copy Markdown

Would be great if this got merged.

I want to make a better appimage release in pkgforge-dev/Anylinux-AppImages#582 since the current appimage being released only works on new glibc systems and still has a libfuse2 dependency...

For that to happen I need the bare binary, the .deb is good enough for this.

@xlc-dev

Copy link
Copy Markdown

+1 for getting this merged. would be awesome

@juliusmarminge

Copy link
Copy Markdown
Member

Closing as part of an open-PR triage — #1655 is being kept as the active candidate for deb packaging (full pipeline). Thanks!

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:unvouchedPR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@Abhishek-Gawade-programmer@Samueru-sama@xlc-dev@juliusmarminge
, '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

Add Ubuntu deb release artifact - #2474

Closed
Abhishek-Gawade-programmer wants to merge 2 commits into
pingdotgg:mainfrom
Abhishek-Gawade-programmer:codex/add-ubuntu-deb-release
Closed

Add Ubuntu deb release artifact#2474
Abhishek-Gawade-programmer wants to merge 2 commits into
pingdotgg:mainfrom
Abhishek-Gawade-programmer:codex/add-ubuntu-deb-release

Conversation

@Abhishek-Gawade-programmer

@Abhishek-Gawade-programmerAbhishek-Gawade-programmer commented May 2, 2026

Copy link
Copy Markdown

What Changed

  • added a dist:desktop:deb script for building a Linux .deb artifact
  • updated the release workflow to build and publish an Ubuntu .deb alongside the existing Linux AppImage artifact
  • updated the release docs to mention the .deb artifact and clarify that Linux auto-update remains AppImage-only

Why

Ubuntu users currently only get the AppImage artifact from the desktop release pipeline.

This keeps the existing AppImage path intact for Linux auto-update behavior, while also publishing a native .deb package for
direct Ubuntu installation. That keeps the change small and avoids changing the current updater assumptions.

Checklist

  • This PR is small and focused
  • I explained what changed and why
  • I included before/after screenshots for any UI changes
  • I included a video for animation/interaction changes

Note

Medium Risk
Modifies the release CI matrix and asset publishing steps; mistakes could break or delay desktop releases, but changes are additive and scoped to packaging/publishing.

Overview
Adds an Ubuntu x64 .deb to the desktop release pipeline alongside the existing Linux AppImage.

The GitHub Actions release.yml build matrix now runs a Linux deb target and the workflow copies/uploads *.deb assets into the GitHub Release. A new dist:desktop:deb script is added for local builds, and release docs are updated to note the new artifact and that Linux auto-update remains AppImage-based.

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

Note

Add Ubuntu x64 .deb artifact to the release build and publish pipeline

  • Adds a Linux Ubuntu x64 .deb target to the build matrix in release.yml, and includes *.deb files in both the file-copy and GitHub release upload steps.
  • Adds a dist:desktop:deb npm script in package.json for local .deb builds.
  • Updates docs/release.md to note that five artifacts are now built, and clarifies that the .deb is for direct installation while AppImage targets auto-update.

Macroscope summarized 009af84.

@coderabbitai

coderabbitaiBot commented May 2, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 293a6421-e8c1-48fd-bf88-fca2993beadc

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actionsgithub-actionsBot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels May 2, 2026
runner: blacksmith-32vcpu-ubuntu-2404
platform: linux
target: deb
arch: x64

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Duplicate artifact name causes release workflow failure

High Severity

The new Ubuntu .deb x64 matrix entry has platform: linux and arch: x64, identical to the existing Linux x64 (AppImage) entry. The artifact upload step on line 357 derives its name from desktop-${{ matrix.platform }}-${{ matrix.arch }}, so both jobs produce desktop-linux-x64. With actions/upload-artifact@v7, the second job to finish will fail because an artifact with that name already exists and no overwrite: true is set. The artifact name needs to incorporate the target to differentiate the two Linux builds.

Additional Locations (1)
Fix in CursorFix in Web

Reviewed by Cursor Bugbot for commit fbd1741. Configure here.

@macroscopeapp

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Needs human review

CI/CD changes to add Ubuntu .deb release artifact. An unresolved high-severity review comment identifies that the new matrix entry would create duplicate artifact names with the existing Linux AppImage build, causing workflow failures. This bug should be addressed before merging.

You can customize Macroscope's approvability policy. Learn more.

@cursorcursorBot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

There are 2 total unresolved issues (including 1 from previous review).

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 009af84. Configure here.

runner: blacksmith-32vcpu-ubuntu-2404
platform: linux
target: deb
arch: x64

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Linux updater metadata collision between AppImage and deb builds

Medium Severity

Both the AppImage and deb builds will generate identically-named auto-update metadata files (latest-linux.yml or nightly-linux.yml) because resolveGitHubPublishConfig injects a publish config for all targets. When artifacts are merged via download-artifact with merge-multiple: true, one metadata file silently overwrites the other. If the deb metadata wins, AppImage auto-update users get pointed to a .deb file, breaking auto-updates. The macOS builds handle a similar multi-arch collision via manifest renaming (lines 335–340), but no equivalent handling exists for the two Linux targets.

Additional Locations (1)
Fix in CursorFix in Web

Reviewed by Cursor Bugbot for commit 009af84. Configure here.

@Samueru-sama

Copy link
Copy Markdown

Would be great if this got merged.

I want to make a better appimage release in pkgforge-dev/Anylinux-AppImages#582 since the current appimage being released only works on new glibc systems and still has a libfuse2 dependency...

For that to happen I need the bare binary, the .deb is good enough for this.

@xlc-dev

Copy link
Copy Markdown

+1 for getting this merged. would be awesome

@juliusmarminge

Copy link
Copy Markdown
Member

Closing as part of an open-PR triage — #1655 is being kept as the active candidate for deb packaging (full pipeline). Thanks!

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:unvouchedPR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@Abhishek-Gawade-programmer@Samueru-sama@xlc-dev@juliusmarminge
, '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

Add Ubuntu deb release artifact - #2474

Closed
Abhishek-Gawade-programmer wants to merge 2 commits into
pingdotgg:mainfrom
Abhishek-Gawade-programmer:codex/add-ubuntu-deb-release
Closed

Add Ubuntu deb release artifact#2474
Abhishek-Gawade-programmer wants to merge 2 commits into
pingdotgg:mainfrom
Abhishek-Gawade-programmer:codex/add-ubuntu-deb-release

Conversation

@Abhishek-Gawade-programmer

@Abhishek-Gawade-programmerAbhishek-Gawade-programmer commented May 2, 2026

Copy link
Copy Markdown

What Changed

  • added a dist:desktop:deb script for building a Linux .deb artifact
  • updated the release workflow to build and publish an Ubuntu .deb alongside the existing Linux AppImage artifact
  • updated the release docs to mention the .deb artifact and clarify that Linux auto-update remains AppImage-only

Why

Ubuntu users currently only get the AppImage artifact from the desktop release pipeline.

This keeps the existing AppImage path intact for Linux auto-update behavior, while also publishing a native .deb package for
direct Ubuntu installation. That keeps the change small and avoids changing the current updater assumptions.

Checklist

  • This PR is small and focused
  • I explained what changed and why
  • I included before/after screenshots for any UI changes
  • I included a video for animation/interaction changes

Note

Medium Risk
Modifies the release CI matrix and asset publishing steps; mistakes could break or delay desktop releases, but changes are additive and scoped to packaging/publishing.

Overview
Adds an Ubuntu x64 .deb to the desktop release pipeline alongside the existing Linux AppImage.

The GitHub Actions release.yml build matrix now runs a Linux deb target and the workflow copies/uploads *.deb assets into the GitHub Release. A new dist:desktop:deb script is added for local builds, and release docs are updated to note the new artifact and that Linux auto-update remains AppImage-based.

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

Note

Add Ubuntu x64 .deb artifact to the release build and publish pipeline

  • Adds a Linux Ubuntu x64 .deb target to the build matrix in release.yml, and includes *.deb files in both the file-copy and GitHub release upload steps.
  • Adds a dist:desktop:deb npm script in package.json for local .deb builds.
  • Updates docs/release.md to note that five artifacts are now built, and clarifies that the .deb is for direct installation while AppImage targets auto-update.

Macroscope summarized 009af84.

@coderabbitai

coderabbitaiBot commented May 2, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 293a6421-e8c1-48fd-bf88-fca2993beadc

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actionsgithub-actionsBot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels May 2, 2026
runner: blacksmith-32vcpu-ubuntu-2404
platform: linux
target: deb
arch: x64

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Duplicate artifact name causes release workflow failure

High Severity

The new Ubuntu .deb x64 matrix entry has platform: linux and arch: x64, identical to the existing Linux x64 (AppImage) entry. The artifact upload step on line 357 derives its name from desktop-${{ matrix.platform }}-${{ matrix.arch }}, so both jobs produce desktop-linux-x64. With actions/upload-artifact@v7, the second job to finish will fail because an artifact with that name already exists and no overwrite: true is set. The artifact name needs to incorporate the target to differentiate the two Linux builds.

Additional Locations (1)
Fix in CursorFix in Web

Reviewed by Cursor Bugbot for commit fbd1741. Configure here.

@macroscopeapp

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Needs human review

CI/CD changes to add Ubuntu .deb release artifact. An unresolved high-severity review comment identifies that the new matrix entry would create duplicate artifact names with the existing Linux AppImage build, causing workflow failures. This bug should be addressed before merging.

You can customize Macroscope's approvability policy. Learn more.

@cursorcursorBot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

There are 2 total unresolved issues (including 1 from previous review).

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 009af84. Configure here.

runner: blacksmith-32vcpu-ubuntu-2404
platform: linux
target: deb
arch: x64

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Linux updater metadata collision between AppImage and deb builds

Medium Severity

Both the AppImage and deb builds will generate identically-named auto-update metadata files (latest-linux.yml or nightly-linux.yml) because resolveGitHubPublishConfig injects a publish config for all targets. When artifacts are merged via download-artifact with merge-multiple: true, one metadata file silently overwrites the other. If the deb metadata wins, AppImage auto-update users get pointed to a .deb file, breaking auto-updates. The macOS builds handle a similar multi-arch collision via manifest renaming (lines 335–340), but no equivalent handling exists for the two Linux targets.

Additional Locations (1)
Fix in CursorFix in Web

Reviewed by Cursor Bugbot for commit 009af84. Configure here.

@Samueru-sama

Copy link
Copy Markdown

Would be great if this got merged.

I want to make a better appimage release in pkgforge-dev/Anylinux-AppImages#582 since the current appimage being released only works on new glibc systems and still has a libfuse2 dependency...

For that to happen I need the bare binary, the .deb is good enough for this.

@xlc-dev

Copy link
Copy Markdown

+1 for getting this merged. would be awesome

@juliusmarminge

Copy link
Copy Markdown
Member

Closing as part of an open-PR triage — #1655 is being kept as the active candidate for deb packaging (full pipeline). Thanks!

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:unvouchedPR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@Abhishek-Gawade-programmer@Samueru-sama@xlc-dev@juliusmarminge
, '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

Add Ubuntu deb release artifact - #2474

Closed
Abhishek-Gawade-programmer wants to merge 2 commits into
pingdotgg:mainfrom
Abhishek-Gawade-programmer:codex/add-ubuntu-deb-release
Closed

Add Ubuntu deb release artifact#2474
Abhishek-Gawade-programmer wants to merge 2 commits into
pingdotgg:mainfrom
Abhishek-Gawade-programmer:codex/add-ubuntu-deb-release

Conversation

@Abhishek-Gawade-programmer

@Abhishek-Gawade-programmerAbhishek-Gawade-programmer commented May 2, 2026

Copy link
Copy Markdown

What Changed

  • added a dist:desktop:deb script for building a Linux .deb artifact
  • updated the release workflow to build and publish an Ubuntu .deb alongside the existing Linux AppImage artifact
  • updated the release docs to mention the .deb artifact and clarify that Linux auto-update remains AppImage-only

Why

Ubuntu users currently only get the AppImage artifact from the desktop release pipeline.

This keeps the existing AppImage path intact for Linux auto-update behavior, while also publishing a native .deb package for
direct Ubuntu installation. That keeps the change small and avoids changing the current updater assumptions.

Checklist

  • This PR is small and focused
  • I explained what changed and why
  • I included before/after screenshots for any UI changes
  • I included a video for animation/interaction changes

Note

Medium Risk
Modifies the release CI matrix and asset publishing steps; mistakes could break or delay desktop releases, but changes are additive and scoped to packaging/publishing.

Overview
Adds an Ubuntu x64 .deb to the desktop release pipeline alongside the existing Linux AppImage.

The GitHub Actions release.yml build matrix now runs a Linux deb target and the workflow copies/uploads *.deb assets into the GitHub Release. A new dist:desktop:deb script is added for local builds, and release docs are updated to note the new artifact and that Linux auto-update remains AppImage-based.

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

Note

Add Ubuntu x64 .deb artifact to the release build and publish pipeline

  • Adds a Linux Ubuntu x64 .deb target to the build matrix in release.yml, and includes *.deb files in both the file-copy and GitHub release upload steps.
  • Adds a dist:desktop:deb npm script in package.json for local .deb builds.
  • Updates docs/release.md to note that five artifacts are now built, and clarifies that the .deb is for direct installation while AppImage targets auto-update.

Macroscope summarized 009af84.

@coderabbitai

coderabbitaiBot commented May 2, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 293a6421-e8c1-48fd-bf88-fca2993beadc

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actionsgithub-actionsBot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels May 2, 2026
runner: blacksmith-32vcpu-ubuntu-2404
platform: linux
target: deb
arch: x64

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Duplicate artifact name causes release workflow failure

High Severity

The new Ubuntu .deb x64 matrix entry has platform: linux and arch: x64, identical to the existing Linux x64 (AppImage) entry. The artifact upload step on line 357 derives its name from desktop-${{ matrix.platform }}-${{ matrix.arch }}, so both jobs produce desktop-linux-x64. With actions/upload-artifact@v7, the second job to finish will fail because an artifact with that name already exists and no overwrite: true is set. The artifact name needs to incorporate the target to differentiate the two Linux builds.

Additional Locations (1)
Fix in CursorFix in Web

Reviewed by Cursor Bugbot for commit fbd1741. Configure here.

@macroscopeapp

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Needs human review

CI/CD changes to add Ubuntu .deb release artifact. An unresolved high-severity review comment identifies that the new matrix entry would create duplicate artifact names with the existing Linux AppImage build, causing workflow failures. This bug should be addressed before merging.

You can customize Macroscope's approvability policy. Learn more.

@cursorcursorBot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

There are 2 total unresolved issues (including 1 from previous review).

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 009af84. Configure here.

runner: blacksmith-32vcpu-ubuntu-2404
platform: linux
target: deb
arch: x64

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Linux updater metadata collision between AppImage and deb builds

Medium Severity

Both the AppImage and deb builds will generate identically-named auto-update metadata files (latest-linux.yml or nightly-linux.yml) because resolveGitHubPublishConfig injects a publish config for all targets. When artifacts are merged via download-artifact with merge-multiple: true, one metadata file silently overwrites the other. If the deb metadata wins, AppImage auto-update users get pointed to a .deb file, breaking auto-updates. The macOS builds handle a similar multi-arch collision via manifest renaming (lines 335–340), but no equivalent handling exists for the two Linux targets.

Additional Locations (1)
Fix in CursorFix in Web

Reviewed by Cursor Bugbot for commit 009af84. Configure here.

@Samueru-sama

Copy link
Copy Markdown

Would be great if this got merged.

I want to make a better appimage release in pkgforge-dev/Anylinux-AppImages#582 since the current appimage being released only works on new glibc systems and still has a libfuse2 dependency...

For that to happen I need the bare binary, the .deb is good enough for this.

@xlc-dev

Copy link
Copy Markdown

+1 for getting this merged. would be awesome

@juliusmarminge

Copy link
Copy Markdown
Member

Closing as part of an open-PR triage — #1655 is being kept as the active candidate for deb packaging (full pipeline). Thanks!

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:unvouchedPR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@Abhishek-Gawade-programmer@Samueru-sama@xlc-dev@juliusmarminge
, '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

Add Ubuntu deb release artifact - #2474

Closed
Abhishek-Gawade-programmer wants to merge 2 commits into
pingdotgg:mainfrom
Abhishek-Gawade-programmer:codex/add-ubuntu-deb-release
Closed

Add Ubuntu deb release artifact#2474
Abhishek-Gawade-programmer wants to merge 2 commits into
pingdotgg:mainfrom
Abhishek-Gawade-programmer:codex/add-ubuntu-deb-release

Conversation

@Abhishek-Gawade-programmer

@Abhishek-Gawade-programmerAbhishek-Gawade-programmer commented May 2, 2026

Copy link
Copy Markdown

What Changed

  • added a dist:desktop:deb script for building a Linux .deb artifact
  • updated the release workflow to build and publish an Ubuntu .deb alongside the existing Linux AppImage artifact
  • updated the release docs to mention the .deb artifact and clarify that Linux auto-update remains AppImage-only

Why

Ubuntu users currently only get the AppImage artifact from the desktop release pipeline.

This keeps the existing AppImage path intact for Linux auto-update behavior, while also publishing a native .deb package for
direct Ubuntu installation. That keeps the change small and avoids changing the current updater assumptions.

Checklist

  • This PR is small and focused
  • I explained what changed and why
  • I included before/after screenshots for any UI changes
  • I included a video for animation/interaction changes

Note

Medium Risk
Modifies the release CI matrix and asset publishing steps; mistakes could break or delay desktop releases, but changes are additive and scoped to packaging/publishing.

Overview
Adds an Ubuntu x64 .deb to the desktop release pipeline alongside the existing Linux AppImage.

The GitHub Actions release.yml build matrix now runs a Linux deb target and the workflow copies/uploads *.deb assets into the GitHub Release. A new dist:desktop:deb script is added for local builds, and release docs are updated to note the new artifact and that Linux auto-update remains AppImage-based.

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

Note

Add Ubuntu x64 .deb artifact to the release build and publish pipeline

  • Adds a Linux Ubuntu x64 .deb target to the build matrix in release.yml, and includes *.deb files in both the file-copy and GitHub release upload steps.
  • Adds a dist:desktop:deb npm script in package.json for local .deb builds.
  • Updates docs/release.md to note that five artifacts are now built, and clarifies that the .deb is for direct installation while AppImage targets auto-update.

Macroscope summarized 009af84.

@coderabbitai

coderabbitaiBot commented May 2, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 293a6421-e8c1-48fd-bf88-fca2993beadc

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actionsgithub-actionsBot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels May 2, 2026
runner: blacksmith-32vcpu-ubuntu-2404
platform: linux
target: deb
arch: x64

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Duplicate artifact name causes release workflow failure

High Severity

The new Ubuntu .deb x64 matrix entry has platform: linux and arch: x64, identical to the existing Linux x64 (AppImage) entry. The artifact upload step on line 357 derives its name from desktop-${{ matrix.platform }}-${{ matrix.arch }}, so both jobs produce desktop-linux-x64. With actions/upload-artifact@v7, the second job to finish will fail because an artifact with that name already exists and no overwrite: true is set. The artifact name needs to incorporate the target to differentiate the two Linux builds.

Additional Locations (1)
Fix in CursorFix in Web

Reviewed by Cursor Bugbot for commit fbd1741. Configure here.

@macroscopeapp

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Needs human review

CI/CD changes to add Ubuntu .deb release artifact. An unresolved high-severity review comment identifies that the new matrix entry would create duplicate artifact names with the existing Linux AppImage build, causing workflow failures. This bug should be addressed before merging.

You can customize Macroscope's approvability policy. Learn more.

@cursorcursorBot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

There are 2 total unresolved issues (including 1 from previous review).

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 009af84. Configure here.

runner: blacksmith-32vcpu-ubuntu-2404
platform: linux
target: deb
arch: x64

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Linux updater metadata collision between AppImage and deb builds

Medium Severity

Both the AppImage and deb builds will generate identically-named auto-update metadata files (latest-linux.yml or nightly-linux.yml) because resolveGitHubPublishConfig injects a publish config for all targets. When artifacts are merged via download-artifact with merge-multiple: true, one metadata file silently overwrites the other. If the deb metadata wins, AppImage auto-update users get pointed to a .deb file, breaking auto-updates. The macOS builds handle a similar multi-arch collision via manifest renaming (lines 335–340), but no equivalent handling exists for the two Linux targets.

Additional Locations (1)
Fix in CursorFix in Web

Reviewed by Cursor Bugbot for commit 009af84. Configure here.

@Samueru-sama

Copy link
Copy Markdown

Would be great if this got merged.

I want to make a better appimage release in pkgforge-dev/Anylinux-AppImages#582 since the current appimage being released only works on new glibc systems and still has a libfuse2 dependency...

For that to happen I need the bare binary, the .deb is good enough for this.

@xlc-dev

Copy link
Copy Markdown

+1 for getting this merged. would be awesome

@juliusmarminge

Copy link
Copy Markdown
Member

Closing as part of an open-PR triage — #1655 is being kept as the active candidate for deb packaging (full pipeline). Thanks!

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:unvouchedPR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@Abhishek-Gawade-programmer@Samueru-sama@xlc-dev@juliusmarminge