Skip to content
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
20 changes: 20 additions & 0 deletions kiichain/proposals/proposal_4.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
{
"messages": [
{
"@type": "/cosmos.upgrade.v1beta1.MsgSoftwareUpgrade",
"authority": "kii10d07y265gmmuvt4z0w9aw880jnsr700jrff0qv",
"plan": {
"name": "v7.1.0-mainnet",
"time": "0001-01-01T00:00:00Z",
"height": "3733250",
Comment on lines +7 to +9
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Description: Compare proposal_3 and proposal_4 to verify version naming pattern

echo "=== Comparing proposal_3 and proposal_4 ==="
echo "Proposal 3:"
cat kiichain/proposals/proposal_3.json | jq -r '{version: .plan.name, height: .plan.height}'

echo -e "\nProposal 4:"
cat kiichain/proposals/proposal_4.json | jq -r '{version: .messages[0].plan.name, height: .messages[0].plan.height}'

echo -e "\n=== All proposals in directory ==="
fd -e json . kiichain/proposals/ --exec sh -c 'echo "File: {}"; cat {} | jq -r ".messages[0].plan.name // .plan.name // \"N/A\"" 2>/dev/null || cat {} | jq -r ".plan.name // \"N/A\"" 2>/dev/null'

Repository: KiiChain/mainnets

Length of output: 467


🏁 Script executed:

echo "=== Proposal 3 Content ===" && cat kiichain/proposals/proposal_3.json && echo -e "\n=== Proposal 4 Content ===" && cat kiichain/proposals/proposal_4.json

Repository: KiiChain/mainnets

Length of output: 1794


Update summary field with correct block height.

Proposal 4 reschedules the same upgrade (v7.1.0-mainnet) for a later block height (3733250), but its summary field still references the original height from proposal 3 (3,585,457). Update the summary to reflect the new scheduled height: "block height 3,733,250" instead of "3,585,457".

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@kiichain/proposals/proposal_4.json` around lines 7 - 9, The proposal's
summary in proposal_4.json is out of date: update the summary text to reference
the new scheduled height from the "height" field ("3733250") so it reads "block
height 3,733,250" instead of "3,585,457"; locate the JSON object for proposal_4
(fields "name", "time", "height") and edit its "summary" value to exactly
reflect the corrected formatted height.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟠 Major

Major: Block height mismatch between plan and summary.

The upgrade plan specifies height "3733250" (line 9), but the summary text incorrectly states "block height 3,585,457" (line 18). The summary appears to have been copied from proposal_3.json without updating the height reference.

📝 Proposed fix
-    "summary": "This proposal upgrades Kiichain to version v7.1.0-mainnet.\n\nThe upgrade is scheduled for 2026-03-19 around 17:00:00 UTC (block height 3,585,457).\n\nChangelog and release details:\n- https://github.com/KiiChain/kiichain/releases/tag/v7.1.0-mainnet",
+    "summary": "This proposal upgrades Kiichain to version v7.1.0-mainnet.\n\nThe upgrade is scheduled for 2026-03-19 around 17:00:00 UTC (block height 3,733,250).\n\nChangelog and release details:\n- https://github.com/KiiChain/kiichain/releases/tag/v7.1.0-mainnet",

Also applies to: 18-18

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@kiichain/proposals/proposal_4.json` at line 9, The summary text in
proposal_4.json references the wrong block height (mentions "3,585,457") while
the upgrade plan's "height" field is "3733250"; update the summary string so any
human-readable height matches the numeric "height" value "3733250" and search
the file for any other occurrences of the old height (e.g., in the summary or
description fields) to replace them so all references are consistent with the
"height" property.

"info": "",
"upgraded_client_state": null
}
}
],
"metadata": "ipfs://CID",
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor

Replace IPFS CID placeholder.

The metadata field contains a placeholder "ipfs://CID" that should be replaced with an actual IPFS content identifier pointing to the full proposal documentation before submission.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@kiichain/proposals/proposal_4.json` at line 15, Replace the placeholder value
for the JSON key "metadata" in proposal_4.json with the actual IPFS content
identifier (e.g., "ipfs://Qm...") that points to the full proposal document;
ensure the value is a valid ipfs://CID string (no extra whitespace or quotes
beyond JSON string rules) so the metadata field resolves to the real proposal
content before submission.

"deposit": "3500000000000000000000akii",
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🔴 Critical

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Description: Verify the expedited_min_deposit parameter from genesis or governance module

# Check governance params in genesis.json
if [ -f "kiichain/genesis.json" ]; then
  echo "=== Expedited Min Deposit from genesis.json ==="
  cat kiichain/genesis.json | jq -r '.app_state.gov.params.expedited_min_deposit'
fi

# Also check if there are any governance parameter update proposals
echo -e "\n=== Searching for governance parameter updates ==="
rg -n "expedited_min_deposit" --type json

Repository: KiiChain/mainnets

Length of output: 294


🏁 Script executed:

cat kiichain/proposals/proposal_4.json | jq -r '.deposit'

Repository: KiiChain/mainnets

Length of output: 87


Critical: Deposit amount insufficient for expedited proposal.

The deposit is 3500000000000000000000akii (3,500 KII), but the expedited minimum deposit requirement is 35000000000000000000000akii (35,000 KII) as defined in genesis.json. This proposal will fail submission as the deposit is 10x too small.

Proposed fix
-    "deposit": "3500000000000000000000akii",
+    "deposit": "35000000000000000000000akii",
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
"deposit": "3500000000000000000000akii",
"deposit": "35000000000000000000000akii",
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@kiichain/proposals/proposal_4.json` at line 16, The "deposit" field in
proposal_4.json is set to "3500000000000000000000akii" which is 3,500 KII and
below the expedited minimum; update the deposit value for the proposal's
"deposit" key to the required expedited minimum "35000000000000000000000akii"
(35,000 KII) so the proposal meets genesis.json expedited deposit requirements.

"title": "Upgrade Kiichain to v7.1.0-mainnet",
"summary": "This proposal upgrades Kiichain to version v7.1.0-mainnet.\n\nThe upgrade is scheduled for 2026-03-19 around 17:00:00 UTC (block height 3,585,457).\n\nChangelog and release details:\n- https://github.com/KiiChain/kiichain/releases/tag/v7.1.0-mainnet",
"expedited": true
}