Revert "feat(rollapp): store all rollapp revisions (#1476)"

This reverts commit 4cad7dc.

app/upgrades/v4/upgrade.go

@@ -361,10 +361,8 @@ func ConvertOldRollappToNew(oldRollapp rollapptypes.Rollapp) rollapptypes.Rollap
 		PreLaunchTime:         nil,                      // We can just let it be zero. Existing rollapps are already launched.
 		LivenessEventHeight:   0,                        // Filled lazily in runtime
 		LastStateUpdateHeight: 0,                        // Filled lazily in runtime
-		Revisions: []rollapptypes.Revision{{
-			Number:      0,
-			StartHeight: 0,
-		}},
+		RevisionNumber:        0,
+		RevisionStartHeight:   0,
 	}
 }
 

proto/dymensionxyz/dymension/rollapp/rollapp.proto

@@ -75,16 +75,10 @@ message Rollapp {
   // received
   int64 last_state_update_height = 18;
 
-  // Revisions is a list of all the rollapp revisions.
-  repeated Revision revisions = 19 [ (gogoproto.nullable) = false ];
-}
-
-// Revision is a representation of the rollapp revision.
-message Revision {
-  // Number is the revision number of the rollapp. Always start with 0 revision.
-  uint64 number = 19;
-  // StartHeight is the first height of the rollapp when the revision started.
-  uint64 start_height = 20;
+  // The revision number of the rollapp. starts always with 0 revision
+  uint64 revision_number = 19;
+  // The first height of the rollapp when the revision started
+  uint64 revision_start_height = 20;
 }
 
 // Rollapp summary is a compact representation of Rollapp

x/lightclient/ante/ibc_msg_update_client.go

@@ -8,11 +8,10 @@ import (
 	sdk "github.com/cosmos/cosmos-sdk/types"
 	ibcclienttypes "github.com/cosmos/ibc-go/v7/modules/core/02-client/types"
 	ibctm "github.com/cosmos/ibc-go/v7/modules/light-clients/07-tendermint"
-	"github.com/dymensionxyz/gerr-cosmos/gerrc"
-
 	"github.com/dymensionxyz/dymension/v3/x/lightclient/types"
 	rollapptypes "github.com/dymensionxyz/dymension/v3/x/rollapp/types"
 	sequencertypes "github.com/dymensionxyz/dymension/v3/x/sequencer/types"
+	"github.com/dymensionxyz/gerr-cosmos/gerrc"
 )
 
 func (i IBCMessagesDecorator) HandleMsgUpdateClient(ctx sdk.Context, msg *ibcclienttypes.MsgUpdateClient) error {
@@ -70,7 +69,7 @@ func (i IBCMessagesDecorator) HandleMsgUpdateClient(ctx sdk.Context, msg *ibccli
 
 	// this disallows LC updates from previous revisions but should be fine since new state roots can be used to prove
 	// state older than the one in the current state root.
-	if header.Header.Version.App != rollapp.LatestRevision().Number {
+	if header.Header.Version.App != rollapp.RevisionNumber {
 		return errorsmod.Wrap(gerrc.ErrFailedPrecondition, "client update revision mismatch")
 	}
 

x/rollapp/keeper/fraud_proposal.go

@@ -6,9 +6,8 @@ import (
 	errorsmod "cosmossdk.io/errors"
 
 	sdk "github.com/cosmos/cosmos-sdk/types"
-	"github.com/dymensionxyz/gerr-cosmos/gerrc"
-
 	"github.com/dymensionxyz/dymension/v3/x/rollapp/types"
+	"github.com/dymensionxyz/gerr-cosmos/gerrc"
 )
 
 // SubmitRollappFraud handles the submission of a fraud proposal

x/rollapp/keeper/hard_fork.go

@@ -5,11 +5,12 @@ import (
 	"sort"
 
 	errorsmod "cosmossdk.io/errors"
+
 	sdk "github.com/cosmos/cosmos-sdk/types"
-	"github.com/dymensionxyz/gerr-cosmos/gerrc"
 
 	common "github.com/dymensionxyz/dymension/v3/x/common/types"
 	"github.com/dymensionxyz/dymension/v3/x/rollapp/types"
+	"github.com/dymensionxyz/gerr-cosmos/gerrc"
 )
 
 // HardFork handles the fraud evidence submitted by the user.
@@ -26,7 +27,8 @@ func (k Keeper) HardFork(ctx sdk.Context, rollappID string, newRevisionHeight ui
 	newRevisionHeight = lastValidHeight + 1
 
 	// update revision number
-	rollapp.BumpRevision(newRevisionHeight)
+	rollapp.RevisionNumber += 1
+	rollapp.RevisionStartHeight = newRevisionHeight
 
 	// stop liveness events
 	k.ResetLivenessClock(ctx, &rollapp)

x/rollapp/keeper/hard_fork_test.go

@@ -125,7 +125,7 @@ func (suite *RollappTestSuite) TestHardFork_AlreadyFinalized() {
 func (suite *RollappTestSuite) assertFraudHandled(rollappId string, height uint64) {
 	rollapp, found := suite.App.RollappKeeper.GetRollapp(suite.Ctx, rollappId)
 	suite.Require().True(found)
-	suite.Require().Equal(uint64(1), rollapp.LatestRevision().Number)
+	suite.Require().Equal(uint64(1), rollapp.RevisionNumber)
 
 	// check states were deleted
 	// the last state should have height less than the fraud height

x/rollapp/keeper/msg_server_create_rollapp_test.go

@@ -202,10 +202,6 @@ func (suite *RollappTestSuite) createRollappWithCreatorAndVerify(
 		VmType:           types.Rollapp_EVM,
 		Metadata:         rollapp.GetMetadata(),
 		GenesisInfo:      *rollapp.GetGenesisInfo(),
-		Revisions: []types.Revision{{
-			Number:      0,
-			StartHeight: 0,
-		}},
 	}
 
 	// create rollapp

x/rollapp/keeper/msg_server_mark_obsolete_rollapps_test.go

@@ -190,9 +190,9 @@ func (s *RollappTestSuite) TestMarkObsoleteRollapps() {
 }
 
 func FilterForked(b types.Rollapp) bool {
-	return b.LatestRevision().Number > 0
+	return b.RevisionNumber > 0
 }
 
 func FilterNonForked(b types.Rollapp) bool {
-	return b.LatestRevision().Number == 0
+	return b.RevisionNumber == 0
 }

x/rollapp/keeper/msg_server_update_state.go

@@ -5,6 +5,7 @@ import (
 
 	errorsmod "cosmossdk.io/errors"
 	sdk "github.com/cosmos/cosmos-sdk/types"
+
 	"github.com/dymensionxyz/gerr-cosmos/gerrc"
 
 	"github.com/dymensionxyz/dymension/v3/x/rollapp/types"
@@ -27,10 +28,10 @@ func (k msgServer) UpdateState(goCtx context.Context, msg *types.MsgUpdateState)
 	}
 
 	// validate correct rollapp revision number
-	if rollapp.LatestRevision().Number != msg.RollappRevision {
+	if rollapp.RevisionNumber != msg.RollappRevision {
 		return nil, errorsmod.Wrapf(types.ErrWrongRollappRevision,
 			"expected revision number (%d), but received (%d)",
-			rollapp.LatestRevision().Number, msg.RollappRevision)
+			rollapp.RevisionNumber, msg.RollappRevision)
 	}
 
 	// retrieve last updating index

x/rollapp/keeper/rollapp_suite_test.go

@@ -74,7 +74,7 @@ func TestRollappKeeperTestSuite(t *testing.T) {
 
 func (suite *RollappTestSuite) assertNotForked(rollappID string) {
 	rollapp, _ := suite.App.RollappKeeper.GetRollapp(suite.Ctx, rollappID)
-	suite.Zero(rollapp.LatestRevision().Number)
+	suite.Zero(rollapp.RevisionNumber)
 }
 
 func (suite *RollappTestSuite) GetRollappLastHeight(rollappID string) uint64 {

x/rollapp/types/rollapp.go

@@ -46,10 +46,8 @@ func NewRollapp(
 		GenesisState: RollappGenesisState{
 			TransfersEnabled: transfersEnabled,
 		},
-		Revisions: []Revision{{
-			Number:      0,
-			StartHeight: 0,
-		}},
+		RevisionNumber:      0,
+		RevisionStartHeight: 0,
 	}
 }
 
@@ -103,22 +101,6 @@ func (r Rollapp) GenesisInfoFieldsAreSet() bool {
 	return r.GenesisInfo.AllSet()
 }
 
-func (r Rollapp) LatestRevision() Revision {
-	if len(r.Revisions) == 0 {
-		// Revision 0 if no revisions exist.
-		// Should happen only in tests.
-		return Revision{}
-	}
-	return r.Revisions[len(r.Revisions)-1]
-}
-
-func (r *Rollapp) BumpRevision(nextRevisionStartHeight uint64) {
-	r.Revisions = append(r.Revisions, Revision{
-		Number:      r.LatestRevision().Number + 1,
-		StartHeight: nextRevisionStartHeight,
-	})
-}
-
 func validateInitialSequencer(initialSequencer string) error {
 	if initialSequencer == "" || initialSequencer == "*" {
 		return nil