you know 2 bug fixes with ai doesnt make it ai generated?#15
Open
SSoggyTacoMan wants to merge 123 commits intorohit1kumar:mainfrom
Open
you know 2 bug fixes with ai doesnt make it ai generated?#15SSoggyTacoMan wants to merge 123 commits intorohit1kumar:mainfrom
SSoggyTacoMan wants to merge 123 commits intorohit1kumar:mainfrom
Conversation
…lation instructions
- Updated .gitignore to include claude-example.html - Revised README.md to reflect Firefox as the primary platform - Changed app/index.html meta tags and titles for Firefox - Updated Footer and Home components to use Globe icon and link to Firefox add-on - Enhanced content.js to support exporting conversations in multiple formats (Markdown, Text, HTML, Word, RTF) - Added popup.html for extension's popup interface with usage instructions - Modified manifest.json to include clipboard permissions and popup settings
Updated installation instructions for Chrome and Firefox, removed duplicate text, and added badge for Chrome platform.
Updated download links for Firefox and added installation instructions.
…e component for replies
…dd web accessible resources in manifest.json
…ck component for improved styling
Updated description to include 'export' functionality.
…ources from manifest.json
… improve message formatting
…y and consistency
…t-14199237705488989684 ⚡ Optimize Plain Text conversion in content.js
Removes a leftover debugging `console.log` from the catch block in `app/functions/api/chats/[id]/raw.js` to improve code health and cleanliness. Functionality remains unaffected.
Added comprehensive test coverage for `splitTextOnExcerpts` and `transformExcerptBlocks` using the native Node.js test runner (`node:test`). The new test suite validates logic against numerous edge cases including missing/empty excerpts, fenced and plain text excerpts, windows style line endings, adjacent excerpts, and placement at beginning/end of strings.
Removed left-over console.log statements from the error handlers in:
- `app/functions/api/chats/[id].js`
- `app/functions/api/chats/index.js`
- `app/functions/api/chats/[id]/raw.js`
In `app/functions/api/chats/index.js`, changed to an optional catch binding `catch { ... }` as the error variable is no longer used, passing ESLint checks.
- Addresses missing error path coverage by mocking drizzle-orm/d1 - Adds full coverage for onRequestGet (200, 404, 500) - Mocks context properly for Cloudflare Pages function testing
Added strict validation for chat API payload in `app/functions/api/chats/index.js`.
Validated that the title is a valid string, and the content is a proper array of
objects. Each object must have a valid `source` ('user' or 'claude') and a string
`message`. Extracted and sanitized the input to avoid inserting extraneous or malicious
keys into the database.
Replace the redundant `.split('|').map().filter()` operations in the markdown table parser with a single-pass regex extraction `/[^|\s](?:[^|]*[^|\s])?/g` and string builder loops. This prevents unnecessary intermediate array creation and iteration, leading to significant CPU/memory improvements.
Replaced the nested loop array push strategy inside convertToDOCX with an inner .map().join('') sequence. This reduces repeated large string reallocations and provides a measured performance improvement for DOCX serialization.
…7148950842503 🧹 [code health improvement] Remove console.debug logging
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
…d-api-7252307406379893356 🧪 [testing improvement] Add error path and coverage for chat retrieval API
…eation-13306378770848422367 🧪 [testing improvement description] Add test coverage for chat creation
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
…alidation-9470193235942884634 🔒 Fix insufficient input validation for chat content
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
…sing-10048407847359540991 ⚡ Optimize markdown table parsing efficiency
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
…-log-12304486679849482427 🧹 [code health] Remove left-over console.log from chat API endpoint
…ole-logs-17949073597137625155 🧹 refactor: remove left-over console.log from API routes
…-2259693742011711880 🧹 code health: remove leftover console.log
…9661571629212737 🧪 add tests for extension text excerpt utilities
…9968988521350451071 ⚡ Optimize DOCX serialization with Array Mapping
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
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.
first of all i fixed the share so now it isnt obsolete cuz llms can read and fetch it and second of all i added an export to document option
if it would have been dead, please archive your repo instead of bitching on other people like some kind of douche bag
https://docs.github.com/en/repositories/archiving-a-github-repository/archiving-repositories