chore(deps): bump rust-toolchain from 1.98.0 to 1.98.1 - #494
chore(deps): bump rust-toolchain from 1.98.0 to 1.98.1#494dependabot[bot] wants to merge 11 commits into
Conversation
Bumps [rust-toolchain](https://github.com/rust-lang/rust) from 1.98.0 to 1.98.1. - [Release notes](https://github.com/rust-lang/rust/releases) - [Changelog](https://github.com/rust-lang/rust/blob/main/RELEASES.md) - [Commits](rust-lang/rust@1.98.0...1.98.1) --- updated-dependencies: - dependency-name: rust-toolchain dependency-version: 1.98.1 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
There was a problem hiding this comment.
Pull request overview
OpenCode could not approve from deterministic current-head evidence because GitHub Checks have failed.
Findings
1. HIGH Current-head GitHub Checks - Fix failed required checks before approval
- Problem: Failed same-head checks remain for
c8acfcfe68fd1f65b62d9f345ad5920adbe3515f. - Root cause: The model-unavailable evidence fallback is allowed only when peer GitHub Checks are complete and clean.
- Fix: Read and fix the failed check logs below, then rerun the current-head checks.
- Regression test: Keep the model-unavailable fallback gated on an empty failed-check rollup.
Failed checks:
- CodeQL PR/CodeQL compatibility analysis (actions): FAILURE (https://github.com/ContextualWisdomLab/TEPP/actions/runs/34303731136/job/102316063867)
- CodeQL PR/CodeQL compatibility analysis (python): FAILURE (https://github.com/ContextualWisdomLab/TEPP/actions/runs/34303731136/job/102316063824)
- CodeQL compatibility analysis (actions) check run: failure (https://github.com/ContextualWisdomLab/TEPP/actions/runs/34303731136/job/102316063867)
- CodeQL compatibility analysis (python) check run: failure (https://github.com/ContextualWisdomLab/TEPP/actions/runs/34303731136/job/102316063824)
- Repository contracts and Python branch coverage check run: failure (https://github.com/ContextualWisdomLab/TEPP/actions/runs/34303730856/job/102315940451)
- Required Noema Review/noema-review: FAILURE (https://github.com/ContextualWisdomLab/TEPP/actions/runs/34303730021/job/102316137265)
- Rust Foundation CI/Repository contracts and Python branch coverage: FAILURE (https://github.com/ContextualWisdomLab/TEPP/actions/runs/34303730856/job/102315940451)
- noema-review check run: failure (https://github.com/ContextualWisdomLab/TEPP/actions/runs/34303730021/job/102316137265)
Changed-File Evidence Map
flowchart LR
PR["PR changed files"] --> Evidence["OpenCode bounded evidence"]
Evidence --> S1["Repository file: rust-toolchain.toml"]
S1 --> I1["repository behavior"]
I1 --> R1["Review risk: Repository file: rust-toolchain.toml"]
R1 --> V1["required checks"]
OpenCode Review Overview
|
|
현재 exact head 이 전환은 source/history를 버리거나 gate를 약화하는 조치가 아닙니다. 기존 Rust 1.98.1 toolchain/CI/scheduler repair delta는 그대로 보존합니다. 다음 owner-correct repair는 (1) 1.98.1은 build-reference pin, |
|
CodeQL RED도 current-head job 단위로 분류했습니다. Run 따라서 이 PR에서 CodeQL requirement를 제거하거나 우회하지 않습니다. Canonical |
|
Fresh foundation coordination from the TEPP Validation Evidence lane: exact The code-current documentation finding in this PR body is still real: |
|
@coderabbitai review |
🧩 Analysis chain🏁 Script executed: #!/bin/bash
set -euo pipefail
echo '== checked-out revision =='
git rev-parse HEAD
git status --short
echo '== candidate tracked files =='
git ls-files | rg '(^|/)(ARCHITECTURE\.md|Cargo\.toml|rust-toolchain\.toml|Dockerfile|README.*|CHANGELOG.*|.*\.(md|mdx|rst|yml|yaml|json|toml))$' || true
echo '== Rust 1.98 version references in tracked text/config files =='
git grep -n -I -E '1\.98\.[01]|rust-version|rust-toolchain|toolchain install|rustup.*install' -- \
':**.md' ':**.mdx' ':**.rst' ':**.toml' ':**.yml' ':**.yaml' ':**.json' \
':Dockerfile' ':**/Dockerfile' ':Cargo.toml' ':**/Cargo.toml' ':rust-toolchain.toml' ':**/rust-toolchain.toml' || true
echo '== architecture and manifest context =='
for f in ARCHITECTURE.md Cargo.toml rust-toolchain.toml Dockerfile; do
if [ -f "$f" ]; then
echo "--- $f ---"
rg -n -C 3 -I -E '1\.98\.[01]|rust-version|toolchain|Rust' "$f" || true
fi
done
echo '== changed files at requested commit relative to first parent =='
git diff-tree --no-commit-id --name-status -r ea2e5ec612b20a05f6a89bb2fc0e896ed1bc4f45^ ea2e5ec612b20a05f6a89bb2fc0e896ed1bc4f45Length of output: 19312 🏁 Script executed: #!/bin/bash
set -euo pipefail
echo '== all tracked 1.98 references with context =='
git grep -n -I -C 3 -E '1\.98\.[01]' -- ':!Cargo.lock' || true
echo '== root manifest and toolchain files =='
cat -n Cargo.toml | sed -n '116,134p'
cat -n rust-toolchain.toml
cat -n ARCHITECTURE.md | sed -n '158,176p'
cat -n docs/research/rust-quality-tooling.md | sed -n '1,105p'
cat -n docs/adr/0007-rust-workspace-quality-gates.md | sed -n '1,85p'
echo '== CI/verifier toolchain and schedule context =='
rg -n -C 5 -I 'rustup toolchain install 1\.98\.1|rust-toolchain\.toml|schedule:|workflow_dispatch|cwl-org-commercial-entrypoint' \
.github/workflows/ci.yml .github/workflows/hourly-nim-product-development.yml
echo '== Python contract references =='
git grep -n -I -C 3 -E 'rust-version|1\.98\.[01]|rust-toolchain' -- '*.py' || true
echo '== cumulative requested-head diff from merge base with main =='
base="$(git merge-base HEAD origin/main 2>/dev/null || git merge-base HEAD main)"
printf 'base=%s\n' "$base"
git diff --stat "$base"..HEAD
git diff --find-renames --unified=40 "$base"..HEAD -- \
rust-toolchain.toml Cargo.toml ARCHITECTURE.md docs/research/rust-quality-tooling.md \
.github/workflows/ci.yml .github/workflows/hourly-nim-product-development.yml \
':**.py'Length of output: 73396
다른 현재 문서와 구성은 이 두 버전을 올바르게 구분합니다.
로컬 스케줄 복원, provider-routing 변경, CodeQL·coverage·security gate 완화, 또는 검증되지 않은 Docker image 변경은 필요하지 않습니다. ✅ Action performedReview finished.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (6)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughRust 툴체인을 1.98.1로 갱신했습니다. 모든 관련 CI 작업과 시간별 제품 품질 게이트가 동일한 버전을 설치합니다. 테스트는 고정 버전과 워크플로 계약을 검증하며, 문서는 변경 근거와 검증 기준을 갱신합니다. ChangesRust 툴체인 정렬
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Suggested reviewers: Merge Risk: ⚪ Minimal · up to This PR only bumps the pinned Rust toolchain from 1.98.1 for CI, coverage, database, and hourly product workflows, keeping the 1.98.0 MSRV unchanged, and updates tests/docs to match. No functional or data-integrity risk was identified in the changed files. The PR remains in draft due to unrelated control-plane/CodeQL receipt issues and pending documentation/approval steps called out in the PR description, not because of a defect found in this review. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Current-head independent review has now converged on the same single repository-local code-current defect: CodeRabbit reviewed I also re-ran only the failed jobs of CodeQL PR |
|
Fresh central-owner handoff: The remaining TEPP-owned #494 source repair is unchanged: correct only the stale |
|
Fresh prerequisite sweep on exact
Keep #494 Draft until the one-line architecture drift is repaired, the central owner yields a terminal authenticated exact-head CodeQL receipt, exact-head checks are reacquired after source mutation, and a qualifying current-head approval exists. No image-tag/digest substitution is authorized without verified immutable 1.98.1 Bookworm evidence. |
|
Fresh exact-source verification still finds one TEPP-local code-current defect on The causal repair is exactly that sentence → The earlier |
Rust 1.98.1 compatibility repair
Rust 1.98.1 was published by the Rust Release Team on 2026-09-03 to fix a rustc vtable-generation miscompilation present in 1.98.0. This Draft keeps the Dependabot patch update and repairs verified repository-level consequences without changing the declared MSRV or weakening CI/security policy.
Current exact head is
67d1c4e3874cf876209459a4d76a472e41102d62on protectedmain@a243f18da4a4ca8a8d068c39922537f1f8ed6ad0.Preserved RED → repair lineage
Original Dependabot head
c8acfcfe68fd1f65b62d9f345ad5920adbe3515fchanged onlyrust-toolchain.tomlfrom 1.98.0 to 1.98.1. Subsequent forward commits synchronized repository contract tests, stable CI lanes, the commercial verifier and current schedule-admission expectations while preservingCargo.toml rust-version = "1.98.0"as MSRV and keeping repository-local scheduling removed.Official compiler source: Rust Release Team. (2026, September 3). Announcing Rust 1.98.1. Rust Blog. https://blog.rust-lang.org/2026/09/03/Rust-1.98.1/
Remaining code-current source finding
ARCHITECTURE.mdstill states: “Stable Rust 1.98.0 is the compile, lint, test, and line-coverage reference.” The branch's root toolchain/CI/verifier build reference is 1.98.1, so that sentence still requires a one-line repair before this PR is code-current.Two narrow-edit attempts have now demonstrated the same whole-file-replacement hazard and are retained as explicit repair evidence rather than hidden. Historical attempt
54ed1dcc6d59cf696763a999a5553ba423ddc372changed the intended sentence but omitted existing architecture-table rows; ordinary-forward97e10a646ca43babcf06c6e36d5bd8f6b1c3b16arestored the exact prior document. A fresh attempt ate44c5ec6ee04f180c16a3314213944b4a8afdd3eagain changed the intended sentence but exact commit diff exposed deletion of an existing repeatedpsychometric_corerow. It was immediately repaired without force by ordinary-forward67d1c4e3874cf876209459a4d76a472e41102d62, which pointsARCHITECTURE.mdback to the exact original blob. Comparing97e10a...to67d1c4e...yields zero file delta. Therefore neither attempted edit is counted as a valid source repair, and no architecture row/data loss remains on the current branch.The remaining acceptance is intentionally strict: apply a byte-preserving one-line patch so that only the stable build-reference sentence changes from 1.98.0 to 1.98.1. Do not globally replace every 1.98.0 occurrence.
Cargo.toml rust-version = "1.98.0"remains the MSRV and historical ADR/CHANGELOG evidence remains historical truth.Dockerfilestill uses the verified immutable 1.98.0 Bookworm image; do not substitute an unverified 1.98.1 tag or digest.The repository-local commercial entrypoint also still contains legacy provider-key/bootstrap behavior owned by #492 /
.github#2038/ contextual-orchestrator #1023/#1083. This compiler patch must not copy or redesign that unpublished orchestration contract.Exact-head verification
The current
67d1c4e...head is source-equivalent to97e10a..., and its fresh workflows are terminal:34684972571: GREEN;34684972550: GREEN;34684972573: GREEN;34684972551: GREEN. Repository contracts/Python branch coverage, format/Clippy/test/rustdoc/dependency policy, Live PostgreSQL, and exact authored line+branch coverage all completed successfully on this head;34684972560: RED only in delegated current-head verdict settlement. Language detection103530276887is GREEN. Python consumer103530319410read the current-head verdict successfully and failed enforcement at 09:10:23Z; actions consumer103530319421did the same at 09:12:53Z. The producer dispatch103531031765did not start until 09:16:55Z and then completed GREEN at 09:17:00Z. This is another exact-head producer-after-consumer canary for the central CodeQL lifecycle defect, not a TEPP compiler defect.The live central repair sequence has advanced beyond the earlier #2051-only description. The bootstrap owner is
.github#2106; the second fresh sweep in this run observed48c6304ffb7294c4748ddc00388e72aa2e9d49bf, open/mergeable and Ready-for-review/Proposed, after an ordinary non-force restack onto protected.github/main@fb17ef556f94f673234aa557254ae52779e9a7b0. Its body records 3,073 passed / 1 skipped / 36 subtests plus the credential-fallback response-isolation repair. Ready is review admission, not merge authorization, and #2106 is still a mutable owner lane whose live head/state must be re-read before acceptance. After the versionedcodeql-scan-v2bootstrap lands normally while preserving v1, canonical producer/consumer successor.github#2040@85522306949bada2b5939608dc911f6374125f1bmust non-force restack and switch its producer to v2. #2051/#2056 remain evidence-preserving predecessors. TEPP must not synthesize a receipt, loop no-op consumer pushes, dropbase_ref, or fork this control-plane repair locally.Review / merge bar
The only formal review remains the historical OpenCode
CHANGES_REQUESTEDsubmission on original headc8acfcfe...; it is not a qualifying current-head approval. There are no unresolved inline review threads. This PR remains Draft and unmerged.Before Ready/merge: repair only the stale
ARCHITECTURE.mdbuild-reference sentence without changing MSRV/history/Docker provenance; wait for the central #2106 → #2040 CodeQL rollout to produce a terminal GREEN receipt on this exact TEPP head; and obtain a qualifying independent current-head approval. No force push, destructive rebase, self-approval, fabricated receipt, unverified image substitution, gate weakening, no-op rerun loop, or routine admin bypass.