-
Notifications
You must be signed in to change notification settings - Fork 0
Bump rain-solmem 0.1.26 -> 0.1.28 and rainlang-interface 0.2.3 -> 0.2.8 #74
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -4,14 +4,14 @@ pragma solidity ^0.8.18; | |
|
|
||
| import {IERC3156FlashLender} from "../../ierc3156/IERC3156FlashLender.sol"; | ||
| //forge-lint: disable-next-line(unused-import) | ||
| import {LibEvaluable} from "rainlang-interface-0.2.3/src/lib/caller/LibEvaluable.sol"; | ||
| import {LibEvaluable} from "rainlang-interface-0.2.8/src/lib/caller/LibEvaluable.sol"; | ||
| import { | ||
| EvaluableConfig, | ||
| Evaluable, | ||
| SignedContext, | ||
| IInterpreterCallerV1 | ||
| } from "rainlang-interface-0.2.3/src/interface/deprecated/v1/IInterpreterCallerV1.sol"; | ||
| import {IExpressionDeployerV3} from "rainlang-interface-0.2.3/src/interface/deprecated/v1/IExpressionDeployerV3.sol"; | ||
| } from "rainlang-interface-0.2.8/src/interface/deprecated/v1/IInterpreterCallerV1.sol"; | ||
| import {IExpressionDeployerV3} from "rainlang-interface-0.2.8/src/interface/deprecated/v1/IExpressionDeployerV3.sol"; | ||
|
Comment on lines
+7
to
+14
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win 🔎 Supported by static analysis🤖 get_repo_knowledge executed:
Length of output: 1850 🏁 Script executed: #!/bin/bash
set -e
printf '%s\n' '--- changed files ---'
git diff --stat
printf '%s\n' '--- relevant diff ---'
git diff -- src/interface/deprecated/v1/IOrderBookV1.sol src/interface/deprecated/v2/IOrderBookV2.sol src/interface/deprecated/v3/IOrderBookV3.sol
printf '%s\n' '--- repository guidance references ---'
rg -n -i --glob '!node_modules' --glob '!dist' 'deprecated|undeprecat|undeprecating|Do not modify deprecated' . | head -200Repository: rainlanguage/raindex.interface Length of output: 1676 🏁 Script executed: #!/bin/bash
set -e
for f in src/interface/deprecated/v1/IOrderBookV1.sol src/interface/deprecated/v2/IOrderBookV2.sol src/interface/deprecated/v3/IOrderBookV3.sol; do
printf '\n--- %s ---\n' "$f"
cat -n "$f" | sed -n '1,80p'
doneRepository: rainlanguage/raindex.interface Length of output: 12876 🏁 Script executed: #!/bin/bash
set -e
printf '%s\n' '--- worktree state ---'
git status --short
printf '%s\n' '--- current commit ---'
git rev-parse HEAD
printf '%s\n' '--- HEAD commit summary ---'
git show --stat --oneline --no-renames HEAD
printf '%s\n' '--- changes in HEAD for the cited files ---'
git diff --no-ext-diff --unified=20 HEAD^ HEAD -- \
src/interface/deprecated/v1/IOrderBookV1.sol \
src/interface/deprecated/v2/IOrderBookV2.sol \
src/interface/deprecated/v3/IOrderBookV3.sol
printf '%s\n' '--- local policy excerpt ---'
sed -n '38,58p' CLAUDE.mdRepository: rainlanguage/raindex.interface Length of output: 7800 🏁 Script executed: #!/bin/bash
set -e
printf '%s\n' '--- dependency version bindings ---'
cat -n remappings.txt
printf '%s\n' '--- lockfile entries for rainlang-interface ---'
rg -n -C 3 'rainlang-interface|0\.2\.[38]' soldeer.lock foundry.toml
printf '%s\n' '--- all deprecated-file changes in HEAD ---'
git diff --no-ext-diff --unified=3 HEAD^ HEAD -- 'src/interface/deprecated/**/*.sol'Repository: rainlanguage/raindex.interface Length of output: 6192 Document an approved exception for the deprecated interface updates. The dependency bump requires these import-path changes because only 📍 Affects 3 files
🤖 Prompt for AI AgentsSource: Coding guidelines |
||
|
|
||
| /// Configuration for a deposit. All deposits are processed by and for | ||
| /// `msg.sender` so the vaults are unambiguous here. | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -6,12 +6,12 @@ import {IERC3156FlashLender} from "../../ierc3156/IERC3156FlashLender.sol"; | |
| import { | ||
| EvaluableConfig, | ||
| Evaluable | ||
| } from "rainlang-interface-0.2.3/src/interface/deprecated/v1/IInterpreterCallerV1.sol"; | ||
| } from "rainlang-interface-0.2.8/src/interface/deprecated/v1/IInterpreterCallerV1.sol"; | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 📐 Maintainability & Code Quality | 🟠 Major | ⚡ Quick win 🔎 Supported by static analysis🤖 get_repo_knowledge executed:
Length of output: 1850 🏁 Script executed: #!/bin/bash
printf '%s\n' '--- changed file and diff ---'
git diff -- src/interface/deprecated/v2/IOrderBookV2.sol
printf '%s\n' '--- relevant source ---'
cat -n src/interface/deprecated/v2/IOrderBookV2.sol | sed -n '1,30p'
printf '%s\n' '--- repository guidance references ---'
rg -n -i --glob '!src/interface/deprecated/v2/IOrderBookV2.sol' 'Do not modify deprecated|deprecated interfaces|undeprecat|approved exception' . | head -80Repository: rainlanguage/raindex.interface Length of output: 1698 🏁 Script executed: #!/bin/bash
git diff -- src/interface/deprecated/v2/IOrderBookV2.sol
cat -n src/interface/deprecated/v2/IOrderBookV2.sol | sed -n '1,25p'
rg -n -i 'deprecated interfaces|Do not modify deprecated|undeprecat' . | head -80Repository: rainlanguage/raindex.interface Length of output: 1340 🤖 get_repo_knowledge executed:
Length of output: 1850 Restore the deprecated interface change unless it is part of an approved undeprecation.
🤖 Prompt for AI AgentsSource: Coding guidelines |
||
| import { | ||
| SignedContextV1, | ||
| IInterpreterCallerV2 | ||
| } from "rainlang-interface-0.2.3/src/interface/deprecated/v1/IInterpreterCallerV2.sol"; | ||
| import {IExpressionDeployerV2} from "rainlang-interface-0.2.3/src/interface/deprecated/v1/IExpressionDeployerV2.sol"; | ||
| } from "rainlang-interface-0.2.8/src/interface/deprecated/v1/IInterpreterCallerV2.sol"; | ||
| import {IExpressionDeployerV2} from "rainlang-interface-0.2.8/src/interface/deprecated/v1/IExpressionDeployerV2.sol"; | ||
|
|
||
| /// Configuration for a deposit. All deposits are processed by and for | ||
| /// `msg.sender` so the vaults are unambiguous here. | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -13,7 +13,7 @@ import { | |
|
|
||
| //forge-lint: disable-next-line(unused-import) | ||
| IInterpreterStoreV2 | ||
| } from "rainlang-interface-0.2.3/src/interface/deprecated/v2/IInterpreterCallerV3.sol"; | ||
| } from "rainlang-interface-0.2.8/src/interface/deprecated/v2/IInterpreterCallerV3.sol"; | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 📐 Maintainability & Code Quality | 🟠 Major | 🏗️ Heavy lift Do not modify deprecated interface files without an undeprecation decision.
As per coding guidelines: 📍 Affects 2 files
🤖 Prompt for AI AgentsSource: Coding guidelines |
||
|
|
||
| /// Import unmodified structures from older versions of `IOrderBook`. | ||
| import {ClearStateChange, ClearConfig, IO} from "../v3/IOrderBookV3.sol"; | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
📐 Maintainability & Code Quality | 🟠 Major | ⚡ Quick win
🔎 Supported by static analysis
🤖 get_repo_knowledge executed:
get_repo_knowledge rainlanguage/raindex.interface /tmp/coderabbit-repo-knowledge/rainlanguage-raindex-interface-130f6c45/conventionsLength of output: 1850
🏁 Script executed:
Repository: rainlanguage/raindex.interface
Length of output: 34765
🏁 Script executed:
Repository: rainlanguage/raindex.interface
Length of output: 34713
🤖 get_repo_knowledge executed:
get_repo_knowledge rainlanguage/raindex.interface /tmp/coderabbit-repo-knowledge/rainlanguage-raindex-interface-130f6c45/conventionsLength of output: 1850
🏁 Script executed:
Repository: rainlanguage/raindex.interface
Length of output: 362
Do not modify this deprecated interface without an approved undeprecation.
src/interface/deprecated/v1/IOrderBookV1.solremains under the protected deprecated path. Restore the imports or document an approved undeprecation before merging.🤖 Prompt for AI Agents
Source: Coding guidelines