Skip to content

Revert "fix(ui): add session management button + fix broken icon paths" - #976

Draft
jelloeater-agent wants to merge 9 commits into
catatafishen:masterfrom
jelloeater-agent:revert-945-fix/sidebar-buttons-layout
Draft

Revert "fix(ui): add session management button + fix broken icon paths"#976
jelloeater-agent wants to merge 9 commits into
catatafishen:masterfrom
jelloeater-agent:revert-945-fix/sidebar-buttons-layout

Conversation

@jelloeater-agent

@jelloeater-agent jelloeater-agent commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Reverts #945

@jelloeater-agent
Revert "chore: remove temporary PR build artifact workflow"
7a81861 
This reverts commit c9a6e97.
@jelloeater-agent
Revert "fix(ui): rename SessionManagementAction to avoid name collisi…
44ff6cc 
…on with RestartSessionGroup"

This reverts commit 0597744.
@jelloeater-agent
Revert "fix(ui): fix power.svg and send.svg icon paths (expui/ prefix)"
b161969 
This reverts commit f193d43.
@jelloeater-agent
Revert "fix(ui): correct icon paths in AgentIconProvider (expui/ pref…
4affaac 
…ix)"

This reverts commit df58f65.
@jelloeater-agent
Revert "fix(ci): add npm ci step for chat-ui build"
3f4aed4 
This reverts commit a854e5a.
@jelloeater-agent
Revert "chore(ci): add PR build artifact workflow"
277672f 
This reverts commit e9f78bc.
@jelloeater-agent
Revert "fix(ui): keep master layout, only add session management button"
23175a6 
This reverts commit 21c900b.
@jelloeater-agent
Revert "fix(ui): add NOWRAP_STRATEGY to controls toolbar so vertical …
6d9daae 
…buttons render"

This reverts commit fe1294e.
@jelloeater-agent
Revert "fix(ui): make sidebar buttons visible and always show session…
926d7c6 
… management"

This reverts commit 86a2d67.
@github-actions

github-actions Bot commented Aug 3, 2026

Copy link
Copy Markdown

⚠️ ClusterFuzzLite — Maintainer Security Review Required

A maintainer needs to approve the ClusterFuzzLite PR fuzzing workflow run for this fork PR.

Before approving, please review the code changes from jelloeater-agent/agentbridge@926d7c62 to confirm there is no code that could:

  • Exfiltrate the CFLITE_STORAGE_REPO_TOKEN CI secret
  • Abuse CI infrastructure or the fuzz corpus storage repository

The fuzz targets from the fork branch will be compiled and executed with access to repository secrets. Rubber-stamping this approval without reviewing the code is a security risk.

This comment is posted automatically whenever ClusterFuzzLite is triggered on a fork PR.

@Jelloeater

Jelloeater commented Aug 3, 2026

Copy link
Copy Markdown

@catatafishen Hey, so sorry this got merges, I should have marked it as WIP, my bad ... it broke icon rendering.

image

@codecov

codecov Bot commented Aug 3, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@jelloeater-agent
jelloeater-agent marked this pull request as draft August 3, 2026 23:43
@Jelloeater

Jelloeater commented Aug 3, 2026

Copy link
Copy Markdown

Ok, looks like it's fine on 1.202.2

image

@Jelloeater

Jelloeater commented Aug 3, 2026

Copy link
Copy Markdown

Also looks fine on 1.203.7 ... weird?
image

image

... guess it was a cached copy?

@Jelloeater

Jelloeater commented Aug 3, 2026

Copy link
Copy Markdown

@catatafishen You seeing this too? or is it just me?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@catatafishen catatafishen Awaiting requested review from catatafishen catatafishen is a code owner

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jelloeater-agent @Jelloeater