Conversation
…ents; change IncludeSubdirectories to false Enhance MainWindowViewModel to initialize tab states and refactor task projections for better performance and clarity Improve App initialization by adding logging and ensuring scheduler setup; handle reactive exceptions more gracefully Update FileStorage to filter out non-JSON files and migration reports when retrieving tasks Refactor FileTaskMigrator to streamline child and parent property handling; improve task migration logic and reporting Modify ServerStorage to use Debug for error logging instead of Console Adjust TaskStorageFactory to disable watcher during initialization for better control Refactor UnifiedTaskStorage to improve task availability checks and migration reporting; add metadata checks for task files
- Updated test classes to replace Xunit with TUnit, changing [Fact] attributes to [Test] and adapting assertions to TUnit syntax. - Modified assertions in StartupProjectionAndRelationsTests, TaskAvailabilityCalculationTests, TaskCompletionChangeTests, TaskMigratorTests, UnifiedTaskStorageMigrationRegressionTests, and other test files for consistency with TUnit. - Removed Xunit package references from the project file and added TUnit package. - Ensured all tests are asynchronous where applicable, improving performance and reliability.
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7e9dbfa5df
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| var issues = issuesToken.ToObject<string[]>() ?? Array.Empty<string>(); | ||
| return issues.Length > 0; |
There was a problem hiding this comment.
ShouldForceReverseLinkRecheck only trusts migration.report content (Version/Issues) and does not compare the report to current task files, so a clean old report can suppress migration even after tasks are replaced (for example after git pull/manual sync). In that case MigrateReverseLinks is skipped and one-sided ContainsTasks/BlocksTasks links remain unhealed, which can leave relations and availability inconsistent on startup.
Useful? React with 👍 / 👎.
1 participant
No description provided.