Skip to content

feat(api): expose logical document deletion through Service API (#44970) - #45017

Closed
hyoban wants to merge 1 commit into
hotfix/1.18.0-fix.2from
hotfix/1.18.0-fix.2-pick-44970
Closed

hyoban wants to merge 1 commit into
hotfix/1.18.0-fix.2from
hotfix/1.18.0-fix.2-pick-44970

Conversation

@hyoban

@hyoban hyoban commented Oct 29, 2026

Copy link
Copy Markdown
Member

Cherry-pick of 4bd3120 (#44970) from main into hotfix/1.18.0-fix.2, created with git cherry-pick -x.

Conflict resolution: packages/contracts/generated/api/service/errors.gen.ts does not exist on the hotfix branch, so it was dropped. I then ran gen-api-contract on the hotfix branch and confirmed that the regenerated contracts exactly match the picked orpc.gen.ts / types.gen.ts / zod.gen.ts, with no errors.gen.ts for service. Git matched the main test test_resources.py to the hotfix test file test_knowledge_fs_service_contracts.py as a rename, so the new test case was added there.

Verification:

  • pytest tests/unit_tests/controllers/test_knowledge_fs_service_contracts.py: 44 passed
  • knowledge-fs-logical-document-delete-contract.smoke.test.ts: 2 passed
  • ruff check / ruff format --check on changed files: clean
  • .github/scripts/check-hotfix-cherry-picks.sh run locally: provenance verified

Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
(cherry picked from commit 4bd3120)
@github-actions github-actions Bot added size:L This PR changes 100-499 lines, ignoring generated files. web This relates to changes on the web. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels Oct 29, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Pyrefly Type Coverage

Metric Base PR Delta
Type coverage 78.32% 78.32% -0.00%
Strict coverage 78.02% 78.02% -0.00%
Typed symbols 73,818 73,822 +4
Untyped symbols 20,511 20,513 +2
Modules 4210 4210 0

@hyoban

hyoban commented Oct 29, 2026

Copy link
Copy Markdown
Member Author

Merged into #45016.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

web This relates to changes on the web.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants