Add Excel and JSON file previews#20314
Closed
GitHamza0206 wants to merge 55 commits intoanomalyco:devfrom
Closed
Conversation
- Add preview-tab-helper.ts with constants and detection helpers - Update preview-tab.tsx to render PDF via embed with base64 data URL - Add pdf to previewExtensions in helpers.ts - Server: return base64 content for previewable binaries (PDF, audio) in File.read - Add previewableBinaryExtensions, isPreviewableBinaryByExtension, getPreviewableBinaryMimeType Made-with: Cursor
…I, and share Made-with: Cursor
Clicking + now shows an Add Project dialog with two options: create a new empty directory (name + location form with native browse on desktop) or open an existing directory. Adds POST /file/mkdir backend endpoint and regenerates the SDK.
…ion preview tab - Removed the download PDF button and its associated logic. - Updated the layout to improve the display of the PDF preview, ensuring the iframe takes up more vertical space.
- Added an upload button in the session side panel for users to upload files. - Introduced a new `write` method in the file handling module to support writing files with base64 encoding. - Updated the backend to handle file write requests via a new POST /file/write endpoint. - Enhanced the SDK to include file writing capabilities.
…amily) Made-with: Cursor
Made-with: Cursor
Made-with: Cursor
Made-with: Cursor
Made-with: Cursor
Made-with: Cursor
Made-with: Cursor
Add /fs/list endpoint for unscoped directory listing and relax workspace middleware to accept non-git directories. Update directory picker dialog to use the new endpoint so users can navigate to any folder on their machine. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Add Integrations popover button in top bar with Pennylane connection status - Add Integrations settings panel with API key configuration - Add Pennylane client import dialog (Add Project > Import from Pennylane) - Add backend endpoints: POST /plugin/pennylane/configure, GET /plugin/pennylane/customers, GET /plugin/pennylane/suppliers - Make upload button more prominent in file tree sidebar - Remove changes tab from right sidebar file tree - Hide Status, Terminal toggle, and Copy path buttons from top bar Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…scripts feat: add Conductor setup and run scripts
- Use `bun run dev` and `bun run dev:web` instead of hardcoded commands - Use CONDUCTOR_PORT for dynamic port allocation (worktree isolation) - Kill stale processes on both API and web ports before starting - Remove nohup/log redirection (output goes directly to conductor) - Remove admin login and lnko workspace registration Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
fix: simplify conductor run.sh with dynamic ports
Resolves SIM-11. Updated dialog labels from project to client terminology and removed the filesystem location picker from the create dialog. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…client-instead-of-new-project-refinend feat: rename New Project to New Client and remove location field
Add a post-creation configuration form for clients with fields for activité principale, régime TVA, présence de salariés, nature des revenus, and règles spécifiques. The form appears automatically after client creation and is re-accessible via right-click context menu. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
File.save() and File.read() prepend Instance.directory, so the path must be relative. Pass the client directory via the directory query parameter instead of embedding it in the file path. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…onfig feat: client config form (SIM-12)
github-actions
bot
added
needs:compliance
Contributor
|
This PR doesn't fully meet our contributing guidelines and PR template. What needs to be fixed:
Please edit this PR description to address the above within 2 hours, or it will be automatically closed. If you believe this was flagged incorrectly, please let a maintainer know. |
Contributor
|
Hey! Your PR title Please update it to start with one of:
Where See CONTRIBUTING.md for details. |
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
2 participants
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.
What does this PR do?
Adds session preview support for JSON and Excel files. JSON files are detected by extension or MIME, formatted for readability, and can still be opened in a new tab. Excel files are now treated as previewable binaries, parsed with
xlsx, and rendered as a simple table with sheet tabs when a workbook has multiple sheets.How did you verify your code works?
Ran
bun test --preload ./happydom.ts ./src/pages/session/preview-tab-helper.test.tsandbun test --preload ./happydom.ts ./src/pages/session/helpers.test.ts. Full pre-push typecheck is currently blocked by an unrelated existing error inpackages/teams/src/index.test.ts(bun:testtypes not found).