Skip to content

Conversation

@lily-de
Copy link
Contributor

@lily-de lily-de commented Mar 19, 2025

Adds an optional display_name field to builtin extensions

this will allow for forwards compatibility with settings v2 and backwards compatibility with the old config style.

Going forward, the name field of built-in extensions should just be the display name and optional display_name field can remain empty.

@lily-de lily-de force-pushed the ldelalande/extension-naming branch from a54737a to e5a9826 Compare March 19, 2025 16:03
@lily-de lily-de changed the title add a display name field extensions: add a display name field Mar 19, 2025
Copy link
Collaborator

@zakiali zakiali left a comment

Choose a reason for hiding this comment

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

lgtm! this worked locally for me through the cli and ui's (existing and alpha)

@lily-de lily-de merged commit 70efaed into main Mar 19, 2025
6 checks passed
@lily-de lily-de deleted the ldelalande/extension-naming branch March 19, 2025 19:42
michaelneale added a commit that referenced this pull request Mar 20, 2025
* main:
  fix: check if working directory has changed before asking (#1733)
  extensions: add a display name field (#1759)
  ui: add logs to app (#1760)
  docs: add stdin (#1769)
  config: add optional extension description (#1743)
  docs: add deployment for install link generator (#1737)
  ui: new configure provider flow (#1736)
  Revert "Standardize Radio Button input" (#1758)
  Settings v2 Add Model (#1708)
  fix: use lowercase names for builtin external extensions (#1756)
ahau-square pushed a commit that referenced this pull request Mar 21, 2025
* origin/main: (74 commits)
  docs: add youtube vscode short (#1809)
  blog: Goose Boston Meetup (#1805)
  docs: Choosing the Right Prompting Style Blog  (#1794)
  docs: Asana tutorial (#1806)
  docs: add langfuse tutorial to blog (#1755)
  docs: Add VS Code tutorial (#1798)
  feat: can read and write to google sheets now (#1784)
  tweak darkmode for modal (#1797)
  docs: asana and gcal blog post (#1796)
  fix: update experimental features docs (#1781)
  docs: Remove Installer Link From Filesystem Tutorial (#1795)
  alexhancock/remove-little-gap (#1793)
  fix: #1562 - Only show text portion of goose message if there is text to display (#1791)
  feat: add /plan command in CLI to invoke reasoner with plan system prompt (#1616)
  docs: Adding Trigger Words to Memory Tutorial (#1783)
  docs: Add prompt completion (#1787)
  fix: check if working directory has changed before asking (#1733)
  extensions: add a display name field (#1759)
  ui: add logs to app (#1760)
  docs: add stdin (#1769)
  ...
ahau-square pushed a commit that referenced this pull request May 2, 2025
cbruyndoncx pushed a commit to cbruyndoncx/goose that referenced this pull request Jul 20, 2025
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.

3 participants