Skip to content

chore: Added goose 2 UI refactor review skill#8903

Merged
morgmart merged 4 commits intomainfrom
lifei/ui-refactor-review-skill
Apr 29, 2026
Merged

chore: Added goose 2 UI refactor review skill#8903
morgmart merged 4 commits intomainfrom
lifei/ui-refactor-review-skill

Conversation

@lifeizhou-ap
Copy link
Copy Markdown
Collaborator

@lifeizhou-ap lifeizhou-ap commented Apr 29, 2026

Summary

This skill will help ui/goose2 contributors to have a sanity check on their prs to make sure code will be easily maintained and extended in the future.

Will recommend the UX designer to try first and if it works well, it can be also referred in the normal code review skill.

Example prompt:
Review my changes (or pr link) for refactoring opportunities

Testing

Tested on #8896 PR

@lifeizhou-ap lifeizhou-ap changed the title chore: Added UI refactor review skill chore: Added goose 2 UI refactor review skill Apr 29, 2026
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 3eab3c261d

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .agents/skills/ui-refactor-review/SKILL.md Outdated
Comment thread .agents/skills/ui-refactor-review/SKILL.md Outdated
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: fe4c8477ae

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .agents/skills/ui-refactor-review/SKILL.md Outdated
Signed-off-by: morgmart <98432065+morgmart@users.noreply.github.com>
@morgmart morgmart added this pull request to the merge queue Apr 29, 2026
Merged via the queue into main with commit 9c16290 Apr 29, 2026
21 checks passed
@morgmart morgmart deleted the lifei/ui-refactor-review-skill branch April 29, 2026 17:55
lifeizhou-ap added a commit that referenced this pull request Apr 30, 2026
* main: (24 commits)
  fix: copy and content improvements in goose2 (#8886)
  feat: make ollama host configurable in goose2 (#8912)
  polish sidebar navigation and project icons (#8896)
  fix: model picker stays usable during provider loading (#8900)
  feat: update provider row after saving credentials (#8914)
  feat: support google model inventory refresh (#8913)
  chore: Added goose 2 UI refactor review skill (#8903)
  blog: goose with peekaboo (#8884)
  blog: Built-in Local Inference blogpost. (#8808)
  perf: parallelize provider resolution and eagerly init SQLite pool (#8899)
  refactor: update goose2 credential management behind provider-scoped ACP/core API (#8887)
  fix: handle acp requests concurrently (#8781)
  build: set LLAMA_STATIC_CRT for Windows CUDA (#8901)
  perf: deduplicate _goose/providers/list RPC call at startup (#8873)
  chore: add a bit more instructions in the release pr (#8890)
  chore: disable spellcheck in model search (#8889)
  add skills to the chat composer (#8881)
  mergeable configs + cleanup (#8378)
  refactor: agent provider to use explicit type states (#8879)
  [goose2] MCP Apps: hydrate and replay app payloads in Goose2 (#8632)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants