-
Notifications
You must be signed in to change notification settings - Fork 876
feat(sidebar): Agents tab + agent logos in terminal tabs #891
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
roberto-fernandino
wants to merge
27
commits into
crynta:main
Choose a base branch
from
roberto-fernandino:feat/agents-sidebar-tab-and-logos
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
27 commits
Select commit
Hold shift + click to select a range
b502645
feat(tabs): select tabs by index within active space
af93248
Merge pull request #1 from roberto-fernandino/fix-tab-picker
roberto-fernandino 023baa1
feat(shortcuts): add ai toggle mini shortcut
bda8dfb
Merge pull request #2 from roberto-fernandino/feat/toggle-ai-chat-win…
roberto-fernandino 2b644ff
feat: terminals in command-palette
1a6b366
Merge pull request #3 from roberto-fernandino/feat/terminals-in-comma…
roberto-fernandino 81e752f
feat(shortcuts): add configurable shortcut to show Files sidebar
df23232
Merge pull request #4 from roberto-fernandino/feat/files-sidebar-shor…
roberto-fernandino 9bf6a2d
chore: add terax-pr-workflow project skill
2a5d3b2
Merge remote-tracking branch 'upstream/main'
roberto-fernandino b679121
feat(sidebar): add Agents tab showing all active AI agent sessions
roberto-fernandino a22273a
Merge pull request #6 from roberto-fernandino/feat/agents-sidebar-tab…
roberto-fernandino bf8faf9
fix(agents): show idle status when agent is at prompt, not working
roberto-fernandino fec2a7d
Merge pull request #7 from roberto-fernandino/fix/agents-idle-status
roberto-fernandino 8a76492
feat(ai): open AI panel by default on startup
roberto-fernandino 74b3f6c
Merge pull request #8 from roberto-fernandino/feat/ai-panel-open-by-d…
roberto-fernandino 497e6b8
fix(agents): clear needs-input when tab focused; finished resets to idle
roberto-fernandino 2f9e17f
Merge pull request #9 from roberto-fernandino/fix/agents-needs-input-…
roberto-fernandino a893d44
fix(agents): reset working status to idle when terminal tab is focused
roberto-fernandino 1b92a83
Merge pull request #10 from roberto-fernandino/fix/agents-working-stu…
roberto-fernandino 14c1ae1
fix(agents): use tabsRef in focus-reset effect to preserve real-time …
roberto-fernandino d292c64
feat(terminal): add cursor style setting (bar/block/underline)
roberto-fernandino f61a0c8
Merge pull request #12 from roberto-fernandino/feat/terminal-cursor-s…
roberto-fernandino 10c67c3
chore: merge upstream/main (sidebar persist, workspace env, Linux cli…
roberto-fernandino e454866
feat(editor): show git blame for current line in status bar
roberto-fernandino 5e7c13c
Merge pull request #13 from roberto-fernandino/feat/git-blame-statusbar
roberto-fernandino c84686d
feat(sidebar): show processes alongside agents
roberto-fernandino File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,130 @@ | ||
| --- | ||
| name: terax-pr-workflow | ||
| description: Use this skill whenever making ANY change to this repository — features, fixes, shortcuts, refactors, anything. It enforces the mandatory branch → commit → dual-PR (fork + upstream) → merge-fork → back-to-main workflow. Trigger on phrases like "create a PR", "add a feature", "fix this", "push this", "open a pull request", or any task that involves committing and shipping code changes. | ||
| user-invocable: true | ||
| --- | ||
|
|
||
| # Terax PR Workflow | ||
|
|
||
| Every change to this repo MUST follow this exact workflow. No exceptions. | ||
|
|
||
| ## Remotes | ||
|
|
||
| - `origin` → `https://github.com/roberto-fernandino/terax-ai-fork.git` (the fork — you have write access) | ||
| - `upstream` → `https://github.com/crynta/terax-ai.git` (the original — NO push access, use cross-fork PRs) | ||
|
|
||
| ## Step-by-step | ||
|
|
||
| ### 1. Create a branch | ||
|
|
||
| ```bash | ||
| git checkout main | ||
| git pull origin main | ||
| git checkout -b feat/<descriptive-name> | ||
| # or fix/<name>, chore/<name>, etc. | ||
| ``` | ||
|
|
||
| Branch name must be kebab-case and describe the change. | ||
|
|
||
| ### 2. Make the changes | ||
|
|
||
| Implement the feature/fix. Type-check before committing: | ||
|
|
||
| ```bash | ||
| npx tsc --noEmit | ||
| ``` | ||
|
|
||
| ### 3. Commit | ||
|
|
||
| Stage only the relevant files (never `git add -A` blindly): | ||
|
|
||
| ```bash | ||
| git add <specific files> | ||
| git commit -m "$(cat <<'EOF' | ||
| feat/fix/chore(scope): short imperative description | ||
|
|
||
| Longer explanation if needed. | ||
|
|
||
| Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> | ||
| EOF | ||
| )" | ||
| ``` | ||
|
|
||
| ### 4. Push to fork | ||
|
|
||
| ```bash | ||
| git push origin <branch-name> | ||
| ``` | ||
|
|
||
| ### 5. Check for existing open PRs — BEFORE creating new ones | ||
|
|
||
| **This step is mandatory.** Before opening any PR, check whether one already exists for the same feature area: | ||
|
|
||
| ```bash | ||
| gh pr list --repo roberto-fernandino/terax-ai-fork --state open | ||
| gh pr list --repo crynta/terax-ai --author roberto-fernandino --state open | ||
| ``` | ||
|
|
||
| - If an open PR exists for the same feature/area → push commits to that branch and update the existing PR. Do NOT create a new one. | ||
| - Only create a new PR if no related open PR exists. | ||
|
|
||
| Related means: same feature, same subsystem, or a fix that belongs to an in-progress feature branch. When in doubt, add to the existing PR. | ||
|
|
||
| ### 6. Create PR on the fork (roberto-fernandino/terax-ai-fork) | ||
|
|
||
| Only if no existing open PR covers this change: | ||
|
|
||
| ```bash | ||
| gh pr create \ | ||
| --repo roberto-fernandino/terax-ai-fork \ | ||
| --base main \ | ||
| --head <branch-name> \ | ||
| --title "<same as commit title>" \ | ||
| --body "$(cat <<'EOF' | ||
| ## Summary | ||
| - bullet points | ||
|
|
||
| ## Test plan | ||
| - [ ] item | ||
|
|
||
| 🤖 Generated with [Claude Code](https://claude.com/claude-code) | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Remove emoji from skill document. The robot emoji violates the project-wide "No emojis anywhere" rule that applies to code, comments, commits, and docs. As per coding guidelines, no emojis anywhere. 🤖 Prompt for AI AgentsSource: Coding guidelines |
||
| EOF | ||
| )" | ||
| ``` | ||
|
|
||
| ### 7. Create cross-fork PR on the upstream (crynta/terax-ai) | ||
|
|
||
| Only if no existing open upstream PR covers this change. Push access is denied to upstream, so use the fork branch as head: | ||
|
|
||
| ```bash | ||
| gh pr create \ | ||
| --repo crynta/terax-ai \ | ||
| --base main \ | ||
| --head roberto-fernandino:<branch-name> \ | ||
| --title "<same as commit title>" \ | ||
| --body "..." | ||
| ``` | ||
|
|
||
| ### 8. Merge the fork PR | ||
|
|
||
| ```bash | ||
| gh pr merge <pr-number> --repo roberto-fernandino/terax-ai-fork --merge --auto | ||
| ``` | ||
|
|
||
| ### 9. Return to main and pull | ||
|
|
||
| ```bash | ||
| git checkout main | ||
| git pull origin main | ||
| ``` | ||
|
|
||
| ## Rules | ||
|
|
||
| - ALWAYS create a branch — never commit directly to main. | ||
| - ALWAYS check for existing open PRs before creating new ones (step 5). Push to the existing branch/PR when the change belongs to the same feature or subsystem. | ||
| - ALWAYS open TWO PRs when creating new ones: one on the fork, one on the upstream (cross-fork). | ||
| - ALWAYS merge the fork PR and return to main at the end. | ||
| - NEVER push directly to upstream (no access); always use `--head roberto-fernandino:<branch>` for the upstream PR. | ||
| - NEVER open a new PR for a fix that belongs to an existing open feature PR — add the commit to that branch instead. | ||
| - Type-check (`npx tsc --noEmit`) before committing. | ||
| - Use conventional commit prefixes: `feat`, `fix`, `chore`, `refactor`, `docs`. | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use
pnpminstead ofnpxfor type-checking.The project conventions mandate pnpm exclusively, never npm/npx. The repo already defines
"check-types": "tsc --noEmit"in package.json and all other documentation (PR template, CONTRIBUTING.md) usespnpm exec tsc --noEmit. Usingnpxhere risks version skew from a globally-installed tsc and breaks workflow consistency. As per coding guidelines, pnpm only, never npm/npx/yarn.🤖 Prompt for AI Agents
Source: Coding guidelines