Commit graph

3 commits

Author SHA1 Message Date
lin onetwo
ee5b48c32a
Fix/misc bug1 (#697)
* fix: move useState/useEffect before conditional early return to fix hooks order error in search mode

* feat: hide friendLinks section from preferences

* feat: add search entry in preferences sidebar that focuses the search input

* feat: remove friendLinks section entirely and split Preference.Search i18n key

* feat: add search entry in workspace sidebar, fix search section title key

* fix: include custom items in search and render info cards in search results

* feat: add missing items to sync/externalAPI/aiAgent schemas for search coverage

* feat: add EmbeddingSection, wire workspace search section, remove old search result views

* fix: apply dark/light palette before serving getIndex to fix startup theme race

* feat: add receiveBundleAndFetch for bundle-based mobile push

* track latest tiddlywiki5 prerelease

* Revert "fix: apply dark/light palette before serving getIndex to fix startup theme race"

This reverts commit e1c096439d.

* Reimplement "fix: apply dark/light palette before serving getIndex to fix startup theme race"

This reverts commit e1c096439d.

* feat: expose runGitCommand, writeTempGitFile, deleteTempGitFile for plugins

Allow TiddlyWiki plugins (like tw-mobile-sync) to run arbitrary git commands
and manage temp files in the .git directory. This reduces the need to rebuild
TidGi Desktop when changing git-related plugin behavior.

Methods include path traversal protection for writeTempGitFile/deleteTempGitFile.

* refactor: replace receiveBundleAndFetch/mergeAfterPush with generic methods

- Removed receiveBundleAndFetch (logic moved to tw-mobile-sync plugin)
- Removed mergeAfterPush (logic moved to tw-mobile-sync plugin)
- Added readWorkspaceFile and writeWorkspaceFile for plugin file I/O
- runGitCommand, writeTempGitFile, deleteTempGitFile remain for plugin git operations
- All mobile-sync-specific logic now lives in the tw-mobile-sync plugin

* test(e2e): deduplicate browser-view palette steps and merge palette scenarios

* test(e2e): isolate palette outline examples by name

* test(e2e): decouple smoke logging assertion from worker lifecycle

* fix: persist workspace immediately on create to survive fast app close

Workspace.create() now calls set() with immediate=true so the new
workspace record is flushed to settings.json synchronously, before any
before-quit handler runs. Previously, if the app was closed while git
init was still running (but after workspace.create had returned), the
workspace record would only be in memory and would be lost on the next
launch, causing Find 0 existing wiki workspaces and a duplicate
default-wiki creation attempt.

Also move applyInitialPaletteBeforeIndexRender from startNodeJSWiki to
IpcServerRoutes.getIndex so the palette is applied on every index
render instead of only at server startup, and thread shouldUseDarkColors
through ipcServerRoutes.setConfig for the same reason.

Rename args param to gitArguments in GitServerService.runGitCommand to
avoid shadowing the outer variable and fix the ESLint no-shadow warning.

* test(e2e): fix browser-view step and wiki-worker restart timing

browserView.ts: replace fixed 10-attempt backoff with a polling loop
that fills the full 25 s Cucumber step budget (210 fast polls), because
WebContentsView creation is async and each failed poll returns in < 1 ms
(no webContents yet), making the old 10-attempt window too narrow.

wiki.ts: replace WIKI_WORKER_STARTED wait before restart with
VIEW_LOADED wait. WIKI_WORKER_STARTED is only written on the direct
startWiki() path; when the app starts via restartWorkspaceViewService
the marker is never emitted. VIEW_LOADED fires after did-stop-loading
and is a reliable proxy that the wiki worker is running and the view is
ready.

application.ts: add launchEnvOverrides map to ApplicationWorld and
thread it into the Electron launch env so scenarios can inject test-only
env vars (e.g. TIDGI_E2E_MOCK_SYSTEM_PALETTE) without modifying the
shared process environment.

* test(e2e): fix @wiki scenario timing and assertions

defaultWiki.feature:
- Explicitly click workspace button before asserting browser-view in
  Background; without a click the WebContentsView is never activated.
- Remove the VIEW_LOADED wait after workspace creation (redundant now
  that browser-view step polls for the full step budget).
- Replace the fragile 'Test content for lazy-all' text assertion with a
  structural check (open tiddler + confirm element exists); in lazy-all
  mode TiddlyWiki renders tiddlers lazily so the body text is often the
  sidebar/chrome rather than the tiddler body at assertion time.
- Remove the 'move back to wiki-test' tail of the move-workspace
  scenario; test-artifacts are wiped before every run so there is no
  need to restore the original location.

simplifiedWiki.feature:
- Wait for WORKSPACE_CREATED log marker before closing the app on first
  launch; git-init inside initWikiGitTransaction can take several
  seconds, and closing too early meant workspace.create() never ran,
  leaving settings.json with no wiki workspace on the second launch.

windowRestore.feature:
- Replace the unreliable WIKI_WORKER_STARTED + VIEW_LOADED wait in
  Background with WORKSPACE_CREATED + 'browser view should be loaded
  and visible'; the former marker is not emitted on the
  restartWorkspaceViewService path used at app startup.

* chore: update pnpm-lock and template/wiki submodule

* Update pnpm-lock.yaml

* test(e2e): remove unused browser view retry constants

* test(e2e): stabilize subwiki update and worker readiness wait

* fix: enforce LF line endings in .gitattributes to match .editorconfig

* feat(e2e): implement CPU-based dynamic timeout scaling

Add cpuBenchmark.ts to measure CPU performance at suite startup and derive
a performance multiplier (≥1.0) that scales all E2E timeouts. Fast machines
get tight timeouts for quick bug detection; slow machines get the room they
need without false timeout failures.

- cpuBenchmark.ts: Run ~200ms pure CPU workload, compare to reference (120ms)
- Local dev: enforce ≥1.8× floor (git I/O overhead not reflected in CPU score)
- CI: keep 1.0× (dedicated runners with known performance)
- timeouts.ts: Apply multiplier to CUCUMBER_GLOBAL_TIMEOUT and derived values
- Print diagnostic: [Timeout Config] multiplier=1.80× step budget=45000 ms

* fix(e2e): fix WorkerServicesReady marker search pattern

The log marker is written as 'test-id-WorkerServicesReady' (without brackets)
but tests were searching for '[test-id-WorkerServicesReady]' (with brackets).
ANSI color codes in logs further broke the pattern match.

Also implement proper marker clearing: wait for initial readiness, restart,
clear stale markers, then wait for fresh markers after restart.

Fixes 3 failing scenarios:
- AI button in Git Log window uses AI-generated commit message
- Plain backup button in Git Log window uses default message
- Move workspace to a new location

* fix(e2e): simplify isWikiRunning check to avoid TypeScript assertion in executeJavaScript

Remove TypeScript type assertion from executeJavaScript string code,
use runtime property checks instead to avoid script execution failure.

* Revert "fix(e2e): fix WorkerServicesReady marker search pattern"

This reverts commit 39f91256092d4ce402eb87b9b70d08c40a6d13fb.

* feat(e2e): increase MIN_MULTIPLIER to 2.0× for local dev

Increase from 1.8× to 2.0× to provide 100% timing cushion for slow
machines. This gives 50s timeout budget (25s base × 2.0) which should
be sufficient for browser view loading and wiki worker operations.

* fix: restore lazy AI deleted test. don't delete test, otherwise I will add it back

* feat(e2e): auto-calibrate timeout using @smoke scenario

Remove MIN_MULTIPLIER hack and synthetic CPU benchmark. Instead:
- @smoke scenario automatically measures real E2E performance
- Hooks record duration and calculate multiplier vs CI baseline
- Pure measurement-based scaling, no hardcoded floors
- Fallback to 3.0× only if smoke hasn't run yet

This captures the full stack (CPU, I/O, Electron startup, rendering) in
one real-world measurement, eliminating guesswork. No separate calibration
command needed - just run tests normally.

* fix(e2e): exclude test files from cucumber require pattern

* fix: should save $:/layout

* chore: update 5.4.0 prerelease

* Update test.yml

* fix: Some repo is clone from github, use https to avoid ssh key issue

delete the lock file and pnpm i regenerate fix this

* lint

* fix(ci): restore search section and precompute E2E timeout calibration

Restore the missing search preferences section so schema validation passes in CI.
Also move E2E timeout calibration ahead of cucumber startup so timeout values
are computed before step modules freeze their constants. Use the smoke scenario
as a real-world baseline and derive a heavier timeout family for restart and
browser-view operations that are dominated by I/O and Electron lifecycle work
on slow machines.

* fix(sync): restore mobile merge-after-push hook

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>

* test(e2e): stabilize selector waits and calibration state

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>

* test(e2e): remove invalid pre-restart worker wait

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>

* fix(preferences): remove global embedding section

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>

* test(e2e): move vector search flow to workspace settings

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>

* fix(vector-search): validate sqlite row ids before storing embeddings

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>

* feat(git-log): add multi-select batch actions

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>

* fix(git-log): support multi-commit undo, fix path traversal, fix search l10n, fix sidebar divider

* fix(test): use data-testid for undo button selector in E2E test

UndoCommit translates to '撤回此提交' in zh-Hans, not '撤销'.
Use data-testid='undo-commit-button' for reliable matching.

* fix(git-log): batch undo via undoCommits, notify git log on tiddler save

- Add undoCommits() to git service: undoes N commits sequentially and fires
  only ONE gitStateChange notification at the end, avoiding the race where
  rapid-fire events drop refreshes due to loadGitLogInProgress guard
- CommitDetailsPanel.handleUndo now uses undoCommits() instead of looping undoCommit()
- FileSystemAdaptor.saveTiddler now calls git.notifyFileChange after every save
  so the git log window auto-refreshes when tiddlers are created/modified, even
  when enableFileSystemWatch is false (the default)

* fix(test): mock git.notifyFileChange in FileSystemAdaptor unit tests

* fix(e2e): use BrowserWindow.id to match tidgiMiniWindow Playwright page

The previous code matched the mini window's Playwright page by document.title,
checking for '太记小窗', 'TidGi Mini Window', or 'TidGiMiniWindow'. But the
mini window loads the same index.html as the main window (title='TidGi'), so the
title-based match fails whenever React hasn't yet updated document.title.

Fix: after finding the BrowserWindow by title (getTitle() contains 'Mini Window')
or by dimensions as a fallback, compare each Playwright Page's underlying
BrowserWindow.id to the found window's id. This is reliable regardless of
document.title state.

* fix(lint): remove unnecessary non-null assertion in application.ts

* fix(lint): rename args to searchParams to satisfy unicorn/prevent-abbreviations

* fix(lint): rename searchParams to searchParameters

* fix(window): wait for isVisible() after showWindow() in test mode

In E2E tests, BrowserWindow.show() is asynchronous with respect to
isVisible() returning true. The IPC call to toggleTidgiMiniWindow()
resolves before the OS has actually marked the window as visible,
causing the subsequent 'confirm visible' step to fail.

Add a poll-until-visible loop (50ms interval) after showWindow()
when isTest is true, so the IPC only resolves once isVisible() is
actually true.

---------

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
2026-04-20 18:42:29 +08:00
lin onetwo
a931058f17
chore: upgrade Electron 39->41 + forge 7.10->7.11 + fix native ABI (#692)
* chore: upgrade Electron 39->41, forge 7.10->7.11, fix native ABI and preload naming

- electron: 39.2.3 -> 41.1.1
- @electron-forge/*: 7.10.2 -> 7.11.1
- better-sqlite3: 12.4.5 -> 12.8.0 (rebuild against Electron 41 ABI 145)
- electron-unhandled: 5.0.0 -> 4.0.1 (v5 uses top-level await, breaks CJS main build)
- vite.preload.config.ts: emit preload.js (not index.js) to avoid collision with main
- viteEntry.ts: getPreloadPath() -> preload.js, renderer path unchanged
- package.json main: .vite/build/main.js (matches forge lib output)
- package.json: strip UTF-8 BOM that broke Volta manifest parsing

* fix: resolve AbortSignal realm mismatch in Electron 41 for streaming tests

In Electron 41, ELECTRON_RUN_AS_NODE mode uses Chromium-based fetch which
requires Blink's AbortSignal. Node.js AbortSignal fails the instanceof check.

Fixes:
- vitest.config.ts: set features/** to node environment via environmentMatchGlobs
  so HTTP-only tests use Node.js fetch (not Chromium fetch)
- setup-vitest.ts: guard all document/window access behind typeof document check
  so setup is safe in both jsdom and node environments
- callProviderAPI.ts: no change needed (signal issue is env-specific)

* ci: increase test timeout to 60min, fix find -o with xargs rm

- timeout-minutes: 25 -> 60 (packaging ~22min + unit tests ~1min + e2e needed)
- fix 'find ... -o ... | xargs rm' -> 'find ... \( -o \) ... -exec rm -rf {} +'
  (the unparenthesized -o with xargs caused 'rm: invalid option -- o' in CI)

* fix: copy tiddlywiki/core-server to packaged output

TiddlyWiki 5.4.0-prerelease introduced core-server/ directory which contains
commander.js (module-type: global). Without it, \.Commander is undefined
when load-modules.js startup runs \.Commander.initCommands(), crashing the
wiki worker with 'Cannot read properties of undefined (reading initCommands)'.

core-server is loaded by boot.js via loadPluginFolder(\.boot.coreServerPath),
which silently returns null when the directory is missing, so the error was
non-obvious and only manifested when actually starting a wiki workspace.

* perf: only screenshot on failure, revert timeout to 25min

AfterStep was capturing a screenshot after every step (~1191 of 1446 steps).
Each screenshot requires IPC round-trip: getFirstWebContentsView  capturePage
 serialize PNG  transfer buffer  fs.writeFile. On CI this costs ~200-400ms
per step, totaling ~6 minutes of pure screenshot overhead.

Now screenshots are only taken for FAILED steps, saving ~6 minutes on CI.
This brings the total test time well within the 25-minute budget.

* fix: flaky tests + merge scenarios to save 8 Electron launches

- crossWindowSync: wait for tiddler to be saved to disk via watch-fs before opening second window
- tidgiMiniWindowWorkspace: add retry backoff to 'should not see elements' step
- tidgiMiniWindow: add explicit window switch after addTiddler
- Merge 5 filesystemPlugin scenarios into 1 (save 4 launches)
- Merge 2 defaultWiki scenarios into 1 (save 1 launch)
- Merge 2 preference scenarios into 1 (save 1 launch)
- Merge smoke + logging into 1 (save 1 launch)
- Merge 2 tiddler scenarios into 1 (save 1 launch)
- Total: 65  57 scenarios (8 fewer Electron launches)

* fix: address CI failures - use IPC for mini window toggle, split talkWithAI clicks

- tidgiMiniWindow: use direct IPC toggle instead of keyboard shortcut after addTiddler
  to avoid race condition between TW syncer and keyboard event dispatch
- talkWithAI: split multi-click step into individual clicks so each gets its own
  25s timeout budget on CI

* fix: await showWindow() in openTidgiMiniWindow to prevent race condition

The two showWindow() calls were using void (fire-and-forget), meaning
toggleTidgiMiniWindow returned before the window was actually shown.
On CI/Xvfb, the test checked isVisible() before show() completed.

* perf: merge scenarios + split CI steps for faster E2E

- Merge 3 defaultWiki scenarios into 1 (save 2 app launches)
- Merge 2 scheduledTask scenarios into 1 (save 1 app launch)
- Split CI test step into unit/prepare/e2e for visibility
- Add pnpm store cache to CI
- Fix viteEntry.ts: remove UTF-8 BOM, update stale JSDoc comment
- Set E2E timeout to 22min (was 25min in single step)

* fix: address Copilot review - main window lookup and comment clarity

- tidgiMiniWindow.ts: use index.html URL pattern to find main window
  (consistent with other step defs, avoids matching preferences window)
- ui.ts: clarify networkidle timeout comment

* fix: use IPC toggle for tidgiMiniWindow sync scenario to avoid flaky keyboard shortcut on CI

---------

Co-authored-by: CI Auto <cidevel@tiddlygit.local>
2026-04-02 21:45:47 +08:00
lin onetwo
fa9751e5ea
Feat/Native AI Agent (#640)
* refactor: only use message id

* feat: stream update, but react don't react on it dny

* feat: start stop

* feat: basic resoponse handlers

* feat: load handler config schema

* chore: upgrade to "moduleResolution": "bundler"

* fix: prompt concat

* feat: rjfs with mui

* fix: mui v7 upgrade

* fix: field editor tabs

* feat: Description field is i18n key, use i18nAlly extension to see it on VSCode. And use react-i18next to translate it on frontend.

* refactor: extract some shared components

* refactor: remove @mui/lab

* feat: faster editor

* feat: beautify editor

* refactor: better style

* fix: fullscreen style

* fix: array sort

* fix: editor not saved

* fix: broken array

* chore: upgrade mui deps

* Update type.d.ts

* feat: upgrade and install react dev tools

* refactor: simplify the code

* refactor: simplify the code

* refactor: simplify ai generated garbage code

* fix: translated label

* feat: translate enum and conditional show field based on enum

* feat: click to open source

* fix: ai generated garbage code to solve id and index problem

* refactor: simplify

* refactor: shorten

* test: add jest

* chore: node-linker=hoisted as electron forge suggested

* test: e2e

* fix: test failed on ci

* test: faster by disable some typecheck

* fix: ci

* fix: ci

* refactor: remove unused

* fix: use macos and take screenshot in ci

* docs:

(cherry picked from commit b1a9706264)

* fix: ci crash due to no dugite binary, due to NODE_ENV=test not set

* Update test.yml

* fix: ci

* Update release.yml

* docs: test

* refactor: move folders and lint

* refactor: organize pages to reduce level

* refactor: merge page service and workspace service to allow drag & drop of pages

* Update ipc-syncadaptor.ts

* Update browserViewMetaData.ts

* fix: view.webContents.loadURL won't reload on new electron version

* fix: try to disable annoying useless disabling pasting "feature"

https://github.com/electron/electron/issues/40995

* fix: initial page

* feat: allow sort add icon

* test: use vitest instead

* refactor: use emotion instead

* fix: all ts error after migrate to vitest and emotion

* fix: Component selectors can only be used in conjunction with @emotion/babel-plugin, the swc Emotion plugin, or another Emotion-aware compiler transform.

fix by not using this usage

* fix: too many open files

* test: add basic main page test

* refactor: split workspace type to make it more specific

* test: support mock react lazy import component by re-export them

* test: sidebar

* refactor: move mock to global

* test: testing library fix

* test: new wiki form

* docs: test with vitest

* lint: fix

* docs: pronounication & remove toc as gh build in

* feat: tools and rag

* feat: prompt for build-in tools

* fix: i18n

* fix: tool using

* test: fix

* refactor: Auto-create default wiki workspace if none exists (handled in backend)

* fix: create wiki workspace so it is on first one

* refactor: remove some useless feature

* refactor: use plugin instead of handler

* chore: make concatPrompt async iterator

* feat: show progress on frontend

* fix: errors

* fix: ConfigError: Config (unnamed): Key "overrides": This appears to be in eslintrc format rather than flat config format.

* Update package.json

* feat: allow windows to hide titlebar

* fix: logger error when ctrl+c on mac

* lint

* refactor: use plugin everywhere

* refactor: move more logic to plugin

* refactor: run tool in plugin, simplify abstraction

* refactor: remove ai generated duplicate

* refactor

* refactor: less plugins

* test: simplify wiki search test

* test: remove any

* fix: not streaming, tool order wrong

* test: plugin system and streaming

* refactor: remove useless auto reply plugin

* fix: hide duration expired tool calling message, so long tool result only show to ai once

* fix: $ props should lower cased

* test: support run as electron so can use sqlite3 compiled bin for electron

* docs: better electron as node run and doc

* test: restore to use threads instead of fock. Seems this also works for inmemory database

* test: fix frontend ui test

* lint: ai content

* test: fix coverage

* test: Refactor test mocks to dedicated __mocks__ directory

Moved common test mocks from setup-vitest.ts into separate files under src/__tests__/__mocks__ for better organization and maintainability. Updated documentation to reflect the new structure. Removed fileMock.js and updated setup-vitest.ts to import and use the new centralized mocks.

* Update ErrorDuringStart.md

* Update messageManagementPlugin.test.ts

* test: Fix Electron test process cleanup and update test config

Update test:unit script to use cross-env for ELECTRON_RUN_AS_NODE, ensuring child processes inherit the variable and preventing resource leaks. Remove manual process.env setting from vitest.config.ts and add documentation on handling related errors in Testing.md. Also, add 'hanging-process' reporter to Vitest config for improved diagnostics.

* fix: warning in test

* Create AgentInstanceWorkflow.md

* fix: duration bug

* fix: ai not response for tool result

* test: Add agent workflow feature and step definitions

Introduces a new Cucumber feature for agent workflow, including multi-round conversation and tool usage. Adds agent-specific step definitions for message input and chat history validation. Refactors application step definitions for improved selector handling and element interaction. Updates test scripts in package.json to include a preparation step for e2e tests.

* Update application.ts

* test: Add mock OpenAI server and tests

Introduces a MockOpenAIServer for simulating OpenAI chat completions, including tool call and tool result responses. Adds corresponding tests and updates vitest config to include tests in the features directory.

* test: simplify steps

* test: Add separate test userData and wiki folders

Introduces 'userData-test' and 'wiki-test' folders for test environments, updating appPaths, fileNames, and paths constants to distinguish between development and test modes. This helps prevent conflicts when running test and development instances simultaneously.

* Update eslint.config.mjs

@typescript-eslint/no-unnecessary-condition': 'off'

* test: Add data-testid attributes for test automation

Added data-testid attributes to form inputs and buttons in ExternalAPI components to improve test automation reliability. Updated feature files and step definitions to use these selectors, refined window switching logic, and improved timing for UI interactions. Also exposed isElectronDevelopment and added isMainWindowPage utility for window identification.

* test: fix wront page type by ai written garbage code

* Update application.ts

* Update Testing.md

* fix: no log during e2e test, and error creating wiki blocks other init steps

* test: click agent workspace button, and refactor mock server

* test: mock streamable openai api

* rename

* chore: try esbuild loader, renderer build too slow

* chore: organize webpack

* chore: ignore service code from hot reload

* Update Testing.md

* test: use full agentinstance in test

* chore: webpack error

* Update Testing.md

* chore: remove useless spectron

* test: EsbuildPlugin's `define` doesn't work, it won't set env properly.

* test: e2e mock openai

* lint: disable @typescript-eslint/require-await

* test: Add quick access to create default agent tab

Introduces a 'Create Default Agent' quick access button in the Agent New Tab page, with localization support. Adds utility to close all tabs and create a default agent tab for fallback scenarios, improves test selectors for tab and close actions, and refactors agent chat tab creation logic for consistency and testability.

* feat: remove unuse favorite

* feat: Add wiki operation and workspaces list plugins

Introduces wikiOperationPlugin and workspacesListPlugin for agent instance prompt and response handling. Updates plugin registry, test coverage, and default agent configuration to support wiki workspace listing and wiki note operations (create, update, delete) via tool calls. Refactors wikiSearchPlugin to delegate workspace list injection to workspacesListPlugin.

* Refactor plugin schema system for dynamic registration

Introduces a dynamic plugin schema registry, allowing plugins to register their parameter schemas and metadata at runtime. Refactors prompt concat schema generation to use dynamically registered plugin schemas, removes static plugin schema definitions, and updates all plugin files to export their parameter schemas. Adds new modelContextProtocolPlugin and schemaRegistry modules, and updates plugin initialization to register schemas and metadata. This enables extensibility and type safety for plugin configuration and validation.

* refactor: move PromptConfigForm to inside PromptPreviewDialog

* test: frontent render tool usage info

* test: split file

* test: fix error

* test: wiki operation

* test: remove log and clean up test deps

* Update i18next-electron-fs-backend.ts

* fix: wikiOperationInServer not called due to no message

* test: fix

* test: Refactor agent feature setup and improve mock server URL handling

Moved AI provider and model configuration to a shared setup scenario in agent.feature to avoid redundant steps in each test. Enhanced application.ts to use a fallback localhost URL for MOCK_SERVER_URL when the mock server is not available, improving test reliability.

* Remove retry logic from basicPromptConcatHandler

Eliminated retryCount and maxRetries from basicPromptConcatHandler, simplifying the control flow and removing the retry limit for LLM calls. Logging and yield logic were updated to reflect the removal of retries.

* Update agent.feature

* test: agent and default wiki

* test: refactor wiki cleanup

* Update agent.feature

* Refactor AI settings setup and add preference feature

Moved AI provider and model configuration steps from agent.feature to a new preference.feature file for better separation of concerns. Updated step definitions in agent.ts to improve robustness when reading and writing settings files, including type usage and directory checks.

* test: try debug can't stop bug

* Update Testing.md

* fix: cancel agent not update cancel button

* refactor: update Frontend use `void window.service.native.log` to log to file.

* test: log from renderer test

* feat: add default embedding model config and victor search service and search preference panel

* test: default embedding form and

* test: default agent

* refator: unused tool listing methods

* Refactor test database setup for integration tests

Centralizes in-memory SQLite test database initialization and cleanup in shared utilities for all integration tests. Updates agentDefinition and messageManagementPlugin tests to use the shared test database, improving reliability and reducing code duplication.

* fix: app path wrong in unit test

* feat: externalAPIDebug

* test: embedding service and let db use real in memory one

* test: provide at least one tab for close tab test

* fix: victor db not loaded in new connection

* test: disable hot reload

* Update DeveloperTools.tsx

* feat: tool message & wiki tool schema

* feat: pref to open db

* chore: skip ExternalsPlugin on dev

* fix: APIs doesn't accept 'tool' role, and it won't return anything when API calls

* fix: docs and remove ai fake fix

* refactor: get agent list don't need to have message

* Update basicPromptConcatHandler.failure.test.ts

* Update basicPromptConcatHandler.test.ts

* fix: role wrong cause e2e failed

* test: allow e2e create .db file to check

* feat: create new agent

* feat: getAgentDefinitionTemplatesFromWikis

* fix: Prevent update non-active (hiding) wiki workspace, so it won't pop up to cover other active agent workspace

* fix: Ensure the config change is fully persisted before proceeding

* test: Don't bring up window when running e2e test, otherwise it will annoy the developer who is doing other things.

* feat: Edit existing agent definition workflow

* Update StyledArrayContainer.tsx

* test: prevent mock server hang

* Refactor UI step definitions into separate file

Moved UI-related Cucumber step definitions from application.ts to a new ui.ts file for better separation of concerns and maintainability. application.ts now only contains application-specific logic.

* lint: ai auto fix

* Clean up feature files and improve test coverage

Removed unnecessary blank lines and improved formatting in feature files for better readability. Updated tsconfig to use 'bundler' module resolution. Enhanced EditAgentDefinitionContent test to mock and verify console.error output. Added type annotation for IAgentDefinitionService in basicPromptConcatHandler.failure.test.ts for improved type safety.

* test: simplify message text match

* feat: transcription and image generation model config

* fix: style props error

* chore: remove unused file

* chore: try to imporove dev start speed but no sig improvment

Replaces 'pnpm start' with 'pnpm start:init' for initial setup and documents slow startup in development. Adds a debug Webpack script, disables polling in renderer file watching, and only enables CircularDependencyPlugin in production/CI for faster dev builds. WebpackBar now only shows when DEBUG includes 'electron-forge:*'. ForkTsCheckerWebpackPlugin is now configured for async checks with memory limits and test file exclusion. Updates documentation to reflect these changes.

* docs: why not vite

* refactor: basic vite usage, but wiki worker is not running and view is not showing

* refactor: remove lazy inject so vite works and wiki worker works, wiki in browser view can loads

* Replace electron-squirrel-startup with inline implementation

Added a custom Squirrel event handler in src/helpers/squirrelStartup.ts to handle Windows install/update/uninstall events, replacing the electron-squirrel-startup package. This avoids ESM/CommonJS compatibility issues and simplifies event handling in src/main.ts.

* refactor: vite build and test

* fix: e2e still use dev wiki folder

* fix: provide env

* chore: "tiddlywiki": "5.3.7"

* Apply suggestion from @Copilot

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Apply suggestion from @Copilot

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Apply suggestion from @Copilot

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Apply suggestion from @Copilot

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* fix: missing i18n

* test:  Module did not self-register: '/home/runner/work/TidGi-Desktop/TidGi-Desktop/node_modules/better-sqlite3/build/Release/better_sqlite3.node'.

* feat: i18n

* feat: zh-Hant i18n

* Update test.yml

* feat: zh-Hans in test

* test: i18n

* refactor: ts forge config

* lint: fix

* chore: update wiki

* Update pnpm-lock.yaml

* chore: update github action versions

* Update wiki

* fix: pnpm then node

* fix: Multiple versions of pnpm specified

* Update test.yml

* fix: Failed to take screenshot: page.screenshot: Target page, context or browser has been closed

* chore: CodeQL Action major versions v1 and v2 have been deprecated.

* chore: parallel codeql

* Update test.yml

* fix: i18n test not passing

* test: step screenshot in each folder

* fix: can't unzip, may due to file path

* test: increase timeout in CI

* docs: more log about wiki creation, and add log to criticial path

* Refactor: logging for structured and consistent output

Replaces string-based logger messages with structured logging throughout the codebase, providing function names, error messages, and relevant context as objects. This improves log readability, enables better filtering and searching, and standardizes error and debug reporting across services.

* chore: debug wiki copy

* fix: wiki submodule not cloned

* Revert "test: increase timeout in CI"

This reverts commit eff8583a01.

* test: reduce wait time, because most check already will wait for a while

* test: batch some e2e steps to reduce screenshot count

* Update index.ts

* chore: remove webpack files, use vite plugins

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
2025-10-10 17:16:56 +08:00