Skip to content

fix(web): tidy the Skills settings header, errors, and switch hints - #18

Merged
donnes merged 1 commit into
mainfrom
t3code/skills-settings-polish
Sep 26, 2026
Merged

donnes merged 1 commit into
mainfrom
t3code/skills-settings-polish

Conversation

@donnes

@donnes donnes commented Sep 26, 2026

Copy link
Copy Markdown
Collaborator

The Skills settings page had a loose count beside a full-width filter, a red "Some skills could not be loaded…" paragraph, and a footnote explaining the switches.

It now uses the same section header as Keybindings: title, then a compact toolbar (count, search, refresh, add). Load errors show as one warning row per environment at the top of the list. The footnote is replaced by a hover hint on each switch ("Shown in / Hidden from skill pickers").

Before:

Screenshot to be attached.

After: screenshot pending.

Done by Claude Opus 5.5 in Claude Code.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Sep 26, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 59 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Repository: codemode-studio/t3code/.coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 13c3bbde-1fe5-4c41-a173-28ebdd1b381e

📥 Commits

Reviewing files that changed from the base of the PR and between 1701041 and cfe2a3c.

📒 Files selected for processing (1)
  • apps/web/src/components/settings/SkillsSettings.tsx

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.

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:L labels Sep 26, 2026
@github-actions

Copy link
Copy Markdown

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire 13.5 KiB 13.5 KiB +1 B (+0.0%) 15.1 KiB ✅
Codex Thread snapshot wire 7.1 KiB 7.1 KiB +5 B (+0.1%) 7.3 KiB ✅
Codex Live turn WebSocket wire 6.5 KiB 6.5 KiB −4 B (−0.1%) 7.8 KiB ✅
Codex Live turn WebSocket decoded 56.3 KiB 56.3 KiB 0 B (0.0%) 66.4 KiB ✅
Codex Live turn messages 10 10 0 (0.0%) 21 ✅
Claude Total thread wire 13.5 KiB 13.6 KiB +17 B (+0.1%) 15.1 KiB ✅
Claude Thread snapshot wire 7.1 KiB 7.1 KiB −1 B (−0.0%) 7.3 KiB ✅
Claude Live turn WebSocket wire 6.5 KiB 6.5 KiB +18 B (+0.3%) 7.8 KiB ✅
Claude Live turn WebSocket decoded 57.1 KiB 57.1 KiB 0 B (0.0%) 66.4 KiB ✅
Claude Live turn messages 10 10 0 (0.0%) 21 ✅

Baseline: 1701041 · PR result: cfe2a3c · 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: 114.0 KiB
  • Claude decoded thread snapshot: 114.7 KiB

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

@donnes
donnes merged commit 3583987 into main Sep 26, 2026
19 checks passed
@donnes
donnes deleted the t3code/skills-settings-polish branch September 26, 2026 11:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L vouch:trusted PR 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