fix: backport upstream PR #359 fixes and infra hardening#1
Merged
Conversation
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…bility Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…eRepositoryDataAsync Upstream PR AIDotNet#359 commit ef36dee renamed the call sites in AdminRepositoryService from ClearRepositoryReferencesAsync to DeleteRepositoryDataAsync but left the method definition with the old name, causing a build error (CS0103). This commit completes the rename. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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.
Summary
8 fix commits on top of
main(e104940), backporting upstream changes from AIDotNet/OpenDeepWiki PR AIDotNet#359 along with local infrastructure and admin UI fixes.What changed
Backend (
src/OpenDeepWiki)AdminRepositoryService: cascade-delete all repository references and switch to soft delete; renameClearRepositoryReferencesAsync→DeleteRepositoryDataAsync(fixes the incomplete rename that produced CS0103 in upstream PR fix: multiple bug fixes and enhancements AIDotNet/OpenDeepWiki#359).SystemSettingDefaults: sync env-var changes into existing system settings on startup so config drift is reflected without manual reset.RepositoryDocsService: URL-decode incoming paths, include directory entries in ZIP export, and drop theFailed-status short-circuit so partial exports succeed.Program.cs: add/healthendpoint for Docker Compose healthcheck.Frontend (
web)Dockerfile: bump Node to 22 forchevrotain@12compatibility.repo-shell/repo-sidebar: consolidate sidebar logic intorepo-shell, decode base64-wrapped ZIP export payload, and fix directory node toggle behavior.Why
Test notes
dotnet build OpenDeepWiki.sln— passes (note: pre-existingTestDbContextgap withIContext.ApiKeysis unrelated to this branch).cd web && npm run build— passes cleanly.npm run lint— pre-existing warnings/errors unrelated to files modified here; PR-modified files contribute zero new lint errors./healthresponds 200, ZIP export of a repository extracts with directory structure intact, and admin repo doc tree expands directories without navigating.Follow-ups
TestDbContextmissingApiKeys(separate branch).