Modernize Clippy with clippyjs library#936
Open
azayrahmad wants to merge 3 commits into
Open
Conversation
- Integrated `clippyjs` as a dependency. - Created `src/apps/clippy/clippy-service.js` to handle agent initialization and port custom extensions (`ask`, `speakAndAnimate`, `speakStream`). - Refactored `src/apps/clippy/clippy.js` to use the new service and support AI response streaming. - Removed legacy scripts and styles from `index.html`. - Deleted `public/clippy/` directory and local assets. - Verified functionality via Playwright E2E tests. Co-authored-by: azayrahmad <10110227+azayrahmad@users.noreply.github.com>
…anup - Integrated `clippyjs` as a dependency. - Created `src/apps/clippy/clippy-service.js` with ported extensions (ask, speakAndAnimate, speakStream). - Refactored `src/apps/clippy/clippy.js` to use `clippyjs` and fixed agent switching logic. - Improved `src/apps/clippy/clippy-app.js` cleanup to properly dispose agents. - Removed legacy assets in `public/clippy/` and script references in `index.html`. Co-authored-by: azayrahmad <10110227+azayrahmad@users.noreply.github.com>
…sues - Replace legacy local Clippy with `clippyjs` npm package. - Centralize agent management and extensions in `src/apps/clippy/clippy-service.js`. - Fix agent switching to properly await hide animation and dispose old agents. - Fix Busy State persistence and clear active states on agent transition. - Fix "Ask Clippy" balloon auto-closing by pausing the hide timeout. - Fix looping animations (e.g., "Thinking") to end when speech completes. - Implement word-by-word streaming for AI responses using `speakStream`. - Remove obsolete local assets and script tags. Co-authored-by: azayrahmad <10110227+azayrahmad@users.noreply.github.com>
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
This change replaces the legacy, manually managed Clippy implementation with the modern
clippyjslibrary. It preserves all custom AzOS features like interactive input balloons and Text-to-Speech integration while adding support for word-by-word streaming of AI responses. Legacy assets inpublic/clippyhave been removed in favor of the npm package.PR created automatically by Jules for task 4353048988539944079 started by @azayrahmad