Conversation
* Update README.zh-CN.md * Update README.zh-CN.md * Update README.zh-CN.md * Update README.md
* Update README.zh-CN.md * Update README.md * Update README.md --------- Co-authored-by: Timothy Carambat <rambat1010@gmail.com>
enable workflow rule
* Update README.zh-CN.md * Update README.zh-CN.md
* wip system prompt history sidebar ui * lint * backend/frontend implementation for prompt history wip * lint * rework ui * add delete menu and delete chat history by id * lint * ref for menu button * reorganize components + light mode styles * lint * UI refactor * backend refactor * remove unused import * add border-none to all buttons * fix spacing on dots 3 icon button * add window to confirm * add english translations * normalize translations * lin * patch store logic * sticky header --------- Co-authored-by: timothycarambat <rambat1010@gmail.com>
* Update common.js * linting --------- Co-authored-by: timothycarambat <rambat1010@gmail.com>
* Update common.js * Update common.js * lint --------- Co-authored-by: timothycarambat <rambat1010@gmail.com>
* Implemented spellcheck toggle in Settings * ran yarn lint * reduce read complexity for appearance spellcheck lint --------- Co-authored-by: timothycarambat <rambat1010@gmail.com>
update:WorkspaceChat ChatContainer PromptInput Co-authored-by: Timothy Carambat <rambat1010@gmail.com>
* Update README.zh-CN.md * Update common.js
* PGVector support for vector db storage * forgot files * comments * dev build * Add ENV connection and table schema validations for vector table add .reset call to drop embedding table when changing the AnythingLLM embedder update instrutions Add preCheck error reporting in UpdateENV add timeout to pg connection * update setup * update README * update doc
* corrected toggle server toast text * reordered code for readability * reordered code for readability * ran yarn lint * reordered code for readability * ran yarn lint
* Update common.js * Update README.md * Update common.js * Update README.md * linting --------- Co-authored-by: timothycarambat <rambat1010@gmail.com>
* add obsidian vault data connector * lint * add english translations * normalize translations * improve file parser and reader --------- Co-authored-by: timothycarambat <rambat1010@gmail.com>
* Fix typos * language --------- Co-authored-by: timothycarambat <rambat1010@gmail.com>
* update light mode designs for system prompt history * lint --------- Co-authored-by: Timothy Carambat <rambat1010@gmail.com>
* Add context window finder from litellm maintained list apply to all cloud providers, have client cache for 3 days * docker container bootup warning * update invalid ENV warning * rebased with current HEAD * update newline printing
* Add context window finder from litellm maintained list apply to all cloud providers, have client cache for 3 days * linting
* fix ux for create api key and create invite * lint --------- Co-authored-by: Timothy Carambat <rambat1010@gmail.com>
* remove summarization from web-scraping flow block * add option to enable/disable summarization in web-scraping block * change layout summarize default to true (was the normal prior) --------- Co-authored-by: Timothy Carambat <rambat1010@gmail.com>
* wip * implment conditional streaming * no bench
…lex-Labs#3838) Respect Max Chunk Len set resolves Mintplex-Labs#3834
* Disable message send while content is embedding * opacity * normalize langs * remove console log --------- Co-authored-by: shatfield4 <seanhatfield5@gmail.com>
* Update README.zh-CN.md * Update common.js * Update common.js
…3853) Filter malformed post-processed files from UI
There was a problem hiding this comment.
Pull Request Overview
This PR syncs the fork with upstream (202505 v2), adds Obsidian vault support, introduces a package‐version verification script with CI, and applies various typo fixes and documentation updates.
- Added Obsidian connector image and backend vault‐loading extension/endpoints
- Introduced
verifyPackageVersions.mjsand corresponding GitHub Action to enforce dependency version parity - Fixed typos in docs/README, updated sponsor and CI workflows
Reviewed Changes
Copilot reviewed 114 out of 117 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| frontend/src/components/Modals/ManageWorkspace/DataConnectors/Connectors/DrupalWiki/index.jsx | Typo fix: “seperated” → “separated” |
| frontend/src/components/LLMSelection/OllamaLLMOptions/index.jsx | Removed initialAuthToken, added masked defaultValue for token input |
| frontend/src/components/LLMSelection/DPAISOptions/index.jsx | New DellProAIStudioOptions component for Dell Pro AI Studio |
| frontend/src/components/DataConnectorOption/media/index.js | Added Obsidian connector image |
| extras/scripts/verifyPackageVersions.mjs | New script to verify package versions between server and collector |
| collector/utils/files/index.js | Exported documentsFolder constant |
| collector/utils/extensions/RepoLoader/GitlabRepo/index.js | Reformatted ternary for wiki description |
| collector/utils/extensions/ObsidianVault/index.js | New Obsidian vault loader extension |
| collector/extensions/index.js | Registered Obsidian vault endpoint |
| cloud-deployments/gcp/deployment/DEPLOY.md | Fixed “one setup” → “once setup” typo |
| cloud-deployments/digitalocean/terraform/DEPLOY.md | Fixed “one setup” → “once setup” typo |
| cloud-deployments/aws/cloudformation/DEPLOY.md | Fixed “one setup” → “once setup” typo |
| README.md | Various typo corrections, added PGVector, sponsors, section count |
| .github/workflows/sponsors.yaml | Added sponsors README generation workflow |
| .github/workflows/dev-build.yaml | Updated dev-build branch filter |
| .github/workflows/check-package-versions.yaml | New workflow to run package version checker |
Files not reviewed (3)
- collector/package.json: Language not supported
- docker/.env.example: Language not supported
- docker/docker-entrypoint.sh: Language not supported
Comments suppressed due to low confidence (2)
frontend/src/components/LLMSelection/OllamaLLMOptions/index.jsx:21
- Removing
initialAuthTokenfrom the auto-discovery hook may prevent the saved token from being used during endpoint discovery. Re-includeinitialAuthToken: settings?.OllamaLLMAuthTokenin the hook options.
// initialAuthToken: settings?.OllamaLLMAuthToken,
.github/workflows/check-package-versions.yaml:36
- The expression
if: failure()isn’t a valid GitHub Actions syntax. Useif: ${{ failure() }}or rely on defaultcontinue-on-errorsettings.
if: failure()
| defaultValue={ | ||
| settings?.OllamaLLMAuthToken ? "*".repeat(20) : "" | ||
| } | ||
| value={authTokenValue.value} |
There was a problem hiding this comment.
Using defaultValue on a controlled input (with value prop) has no effect. Consider initializing the state in your hook or using a placeholder instead of defaultValue.
| defaultValue={ | |
| settings?.OllamaLLMAuthToken ? "*".repeat(20) : "" | |
| } | |
| value={authTokenValue.value} | |
| value={authTokenValue.value || (settings?.OllamaLLMAuthToken ? "*".repeat(20) : "")} |
| <option | ||
| key={model.id} | ||
| value={model.id} | ||
| selected={settings.DellProAiStudioModelPref === model.id} |
There was a problem hiding this comment.
Inline selected attributes on <option> can cause React warnings. It’s better to control the <select> by giving it a value prop tied to state or settings.
| selected={settings.DellProAiStudioModelPref === model.id} |
| * The folder where documents are stored to be stored when | ||
| * processed by the collector. |
There was a problem hiding this comment.
[nitpick] The comment reads “stored to be stored,” which is redundant. Consider rephrasing to something like “The folder where documents are placed for processing by the collector.”
| * The folder where documents are stored to be stored when | |
| * processed by the collector. | |
| * The folder where documents are placed for processing by the collector. |
| } | ||
| ); | ||
|
|
||
| app.post( |
There was a problem hiding this comment.
[nitpick] You’ve added a new /ext/obsidian/vault endpoint—consider updating any API documentation or OpenAPI specs so consumers know about this new route.
| on: | ||
| push: | ||
| branches: ['2237-auto-stt-submit-preference'] # put your current branch to create a build. Core team only. | ||
| branches: ['warn-bad-docker-command'] # put your current branch to create a build. Core team only. |
There was a problem hiding this comment.
[nitpick] The dev-build workflow is hard-coded to run only on warn-bad-docker-command. This may be out of date—consider parameterizing or documenting this filter to avoid dangling configurations.
Pull Request Type
Relevant Issues
resolves #xxx
What is in this change?
Additional Information
Developer Validations
yarn lintfrom the root of the repo & committed changes