fix: make ACK manifest checks cross-platform
This commit is contained in:
1 parent
bb004f02b3
commit
79d29bbf31
2 files changed
+15
-4
No files matched your search
@@ -8,7 +8,7 @@
|
|||||||
- Worktree: D:\Datas\OthersProjects\NianAIGC-integration-rds-4e61c2
|
- Worktree: D:\Datas\OthersProjects\NianAIGC-integration-rds-4e61c2
|
||||||
- Base commit: 84d84ba94f136f10624700e8d34ed19fc6fe7fe7
|
- Base commit: 84d84ba94f136f10624700e8d34ed19fc6fe7fe7
|
||||||
- Owner: codex
|
- Owner: codex
|
||||||
- Status: Planning
|
- Status: Ready for Integration
|
||||||
|
|
||||||
## Scope
|
## Scope
|
||||||
|
|
||||||
@@ -25,16 +25,27 @@
|
|||||||
|
|
||||||
## Outcome
|
## Outcome
|
||||||
|
|
||||||
- Integration in progress.
|
- Committed the initializer-created project-memory baseline and completed Docker verification record on `main`, then released the previously occupied main worktree without deleting unknown files.
|
||||||
|
- Rebased the reviewed RDS/ACK implementation onto the updated `main`; `git range-diff` confirmed the implementation patch was unchanged.
|
||||||
|
- Promoted the source task's accepted backend, migration, workload, and operational boundaries into canonical project memory.
|
||||||
|
- Corrected the ACK manifest checker to normalize CRLF before its multiline assertion, making the verified deployment check portable on Windows.
|
||||||
|
|
||||||
## Verification
|
## Verification
|
||||||
|
|
||||||
- Source feature commit `84d84ba` is present and based directly on `main` through initialization commit `0bcb149`.
|
- Source feature commit `84d84ba` is present and based directly on `main` through initialization commit `0bcb149`.
|
||||||
- Source task final and incremental Sol reviews returned `PASS`.
|
- Source task final and incremental Sol reviews returned `PASS`.
|
||||||
|
- Integration verification found and corrected a Windows CRLF sensitivity in `scripts/check-ack-manifests.mjs`; manifest contents were valid, but the assertion used a literal LF-only substring.
|
||||||
|
- `npm ci --ignore-scripts` passed in the final integration worktree.
|
||||||
|
- `npm test -- --run` passed: 31 files / 119 tests.
|
||||||
|
- `npx tsc --noEmit --pretty false --incremental false` passed.
|
||||||
|
- `npm run deploy:check` passed: 8 ACK manifests.
|
||||||
|
- `pnpm install --lockfile-only --frozen-lockfile` passed.
|
||||||
|
- `npm run build` passed and includes `/api/ready`.
|
||||||
|
- Live RDS migration, Docker container startup, and ACK rollout remain unverified because those external services were unavailable.
|
||||||
|
|
||||||
## Follow-ups
|
## Follow-ups
|
||||||
|
|
||||||
- None recorded.
|
- Use the deployment checklist recorded in canonical current state before production cutover.
|
||||||
|
|
||||||
## Promotion Candidates
|
## Promotion Candidates
|
||||||
|
|
||||||
|
|||||||
@@ -29,7 +29,7 @@ assert(service.includes("name: zhinian-public-deny"), "selectorless deny Service
|
|||||||
console.log(`ACK manifest assertions passed (${files.length} files)`);
|
console.log(`ACK manifest assertions passed (${files.length} files)`);
|
||||||
|
|
||||||
function read(file) {
|
function read(file) {
|
||||||
return readFileSync(new URL(file, directory), "utf8");
|
return readFileSync(new URL(file, directory), "utf8").replace(/\r\n/g, "\n");
|
||||||
}
|
}
|
||||||
|
|
||||||
function assert(condition, message) {
|
function assert(condition, message) {
|
||||||
|
|||||||
Reference in new issue
Block a user