docs(integration): record Learning download promotion
This commit is contained in:
1 parent
399b28097f
commit
a5a3205848
2 files changed
+59
-72
No files matched your search
@@ -1,72 +0,0 @@
|
|||||||
# Task: Integrate Learning download size removal
|
|
||||||
|
|
||||||
## Identity
|
|
||||||
|
|
||||||
- Task ID: 20260820-integrate-download-size-6e3a91c2
|
|
||||||
- Mode: Integration
|
|
||||||
- Branch: codex/20260820-integrate-download-size-6e3a91c2-integrate-download-size
|
|
||||||
- Worktree: D:\Datas\OthersProjects\makelore-integrate-download-size-6e3a91c2
|
|
||||||
- Base commit: 2168e291b2bf8ac8690c482f56400554a8d77531
|
|
||||||
- Owner: codex
|
|
||||||
- Status: Ready for Integration
|
|
||||||
|
|
||||||
## Scope
|
|
||||||
|
|
||||||
- Merge verified source commit `8509084` into the integration branch.
|
|
||||||
- Promote the accepted Learning download behavior into ADR-005, architecture,
|
|
||||||
domain rules, success criteria, README, server contract, current state,
|
|
||||||
evidence, and the pending production-smoke commitment.
|
|
||||||
- Advance local `main` without overwriting its pre-existing untracked task
|
|
||||||
record.
|
|
||||||
|
|
||||||
## Intent And Constraints
|
|
||||||
|
|
||||||
- Follow the user's explicit decision that project downloads must not perform
|
|
||||||
archive-size validation.
|
|
||||||
- Preserve Main ownership of authentication, fixed archive routes, same-origin
|
|
||||||
redirects, native destination selection, SHA-256, ZIP signature, temporary
|
|
||||||
files, cleanup, and atomic rename.
|
|
||||||
- Keep the Works Square upload/storage size policy distinct from client download
|
|
||||||
behavior: the service may retain an independent operations upload limit, but
|
|
||||||
Makelore does not enforce it while downloading.
|
|
||||||
- Do not modify, add, remove, or adopt the unrelated untracked record already in
|
|
||||||
the canonical `main` worktree.
|
|
||||||
|
|
||||||
## Outcome
|
|
||||||
|
|
||||||
- Merged source commit `8509084` through integration merge `0c1a360`.
|
|
||||||
- Promoted the no-size-validation behavior into ADR-005, the decision index,
|
|
||||||
system overview, module map, data flow, domain rules, glossary, success
|
|
||||||
criteria, current state, evidence index, README, server collaboration
|
|
||||||
contract, and Learning production-smoke commitment.
|
|
||||||
- Kept the optional 512 MiB rule only as an independent Works Square operations
|
|
||||||
upload/storage policy; Makelore does not enforce it during download.
|
|
||||||
- Preserved SHA-256, ZIP signature, same-origin redirects, account binding,
|
|
||||||
temporary-file cleanup, and atomic save.
|
|
||||||
|
|
||||||
## Verification
|
|
||||||
|
|
||||||
- Pinned pnpm 10.33.4 frozen install: passed.
|
|
||||||
- Focused Learning tests: `2` files, `17 passed`.
|
|
||||||
- Full unit suite: first run had one unrelated Robot async-loading test failure;
|
|
||||||
that test passed immediately in isolation, and the complete rerun passed
|
|
||||||
`176` files / `2061` tests.
|
|
||||||
- `pnpm run typecheck`: passed.
|
|
||||||
- `pnpm run lint:check`: passed with zero errors and six pre-existing warnings.
|
|
||||||
- `pnpm run build:vite`: passed for Renderer, Main, Preload, and utility worker;
|
|
||||||
existing dynamic-import and chunk-size advisories only.
|
|
||||||
- Canonical size-rule scan and `git diff --check`: passed.
|
|
||||||
- Independent sub-agent review was not run because the user explicitly required
|
|
||||||
that no sub-agents be created; final integration review stayed in the primary
|
|
||||||
task.
|
|
||||||
|
|
||||||
## Follow-ups
|
|
||||||
|
|
||||||
- Package and smoke the merged client with a real archive response that omits or
|
|
||||||
misreports `Content-Length`.
|
|
||||||
- Monitor disk-space behavior for very large archives because the client now has
|
|
||||||
no archive-size ceiling by explicit product decision.
|
|
||||||
|
|
||||||
## Promotion Candidates
|
|
||||||
|
|
||||||
- None; accepted source-task candidates were promoted in this Integration Gate.
|
|
||||||
@@ -0,0 +1,59 @@
|
|||||||
|
# Task: Promote Learning download size removal to main
|
||||||
|
|
||||||
|
## Identity
|
||||||
|
|
||||||
|
- Task ID: 20260820-promote-download-size-main-9a4c2e71
|
||||||
|
- Mode: Integration
|
||||||
|
- Branch: main
|
||||||
|
- Worktree: D:\Datas\OthersProjects\makelore
|
||||||
|
- Base commit: 2168e291b2bf8ac8690c482f56400554a8d77531
|
||||||
|
- Owner: codex
|
||||||
|
- Status: Ready for Integration
|
||||||
|
|
||||||
|
## Scope
|
||||||
|
|
||||||
|
- Promote the already verified integration tip `d16cace` into local `main`.
|
||||||
|
- Keep the reconciled Learning download implementation and canonical project
|
||||||
|
documentation unchanged.
|
||||||
|
- Preserve the unrelated pre-existing untracked task record in the canonical
|
||||||
|
worktree without modifying or committing it.
|
||||||
|
|
||||||
|
## Intent And Constraints
|
||||||
|
|
||||||
|
- Remove every client-side archive-size check while retaining authentication,
|
||||||
|
fixed routes, same-origin redirects, SHA-256, ZIP signature validation,
|
||||||
|
temporary-file cleanup, and atomic rename.
|
||||||
|
- Treat the Works Square upload/storage limit as independent from Makelore's
|
||||||
|
download behavior.
|
||||||
|
- Merge only to local `main`; do not push a remote branch.
|
||||||
|
- Do not create sub-agents, per the user's explicit instruction.
|
||||||
|
|
||||||
|
## Outcome
|
||||||
|
|
||||||
|
- Merged integration tip `d16cace` into local `main` through merge commit
|
||||||
|
`399b280`.
|
||||||
|
- Removed the completed upstream integration task record from the promoted tree
|
||||||
|
so the canonical worklog contains only the current promotion record.
|
||||||
|
- Preserved `.project-docs/30-worklog/tasks/20260819-package-learning-off-115-9c4d.md`
|
||||||
|
untouched and untracked.
|
||||||
|
|
||||||
|
## Verification
|
||||||
|
|
||||||
|
- Upstream integration verification: focused Learning tests `17 passed`; full
|
||||||
|
unit suite `176` files / `2061` tests; typecheck passed; lint passed with zero
|
||||||
|
errors and six pre-existing warnings; Renderer/Main/Preload/utility build
|
||||||
|
passed with existing advisory warnings only.
|
||||||
|
- Local `main` focused Learning tests: `2` files / `17 passed`.
|
||||||
|
- Local `main` `pnpm run typecheck`: passed.
|
||||||
|
- Removed-size-symbol scan and `git diff --check`: passed.
|
||||||
|
|
||||||
|
## Follow-ups
|
||||||
|
|
||||||
|
- Package and smoke with a real archive response that omits or misreports
|
||||||
|
`Content-Length`.
|
||||||
|
- Monitor disk-space behavior for very large archives because the client has no
|
||||||
|
archive-size ceiling by explicit product decision.
|
||||||
|
|
||||||
|
## Promotion Candidates
|
||||||
|
|
||||||
|
- None recorded.
|
||||||
Reference in new issue
Block a user