Fix logic to add expiration to delegation

packages/account-postgres-sink-service/vote_service_example.json

@@ -29,6 +29,18 @@
           "schema": "public"
         }
       ]
+    },
+    {
+      "programId": "hdaoVTCqhfHHo75XdAMxBKdUqvq1i5bF23sisBqVgGR",
+      "accounts": [
+        { "type": "SubDaoV0", "table": "sub_daos", "schema": "public" },
+        { "type": "DaoV0", "table": "daos", "schema": "public" },
+        {
+          "type": "DelegatedPositionV0",
+          "table": "delegated_positions",
+          "schema": "public"
+        }
+      ]
     }
   ],
   "indexConfigs": []

packages/helium-admin-cli/src/add-expiration-to-delegations.ts

@@ -51,11 +51,11 @@ export async function run(args: any = process.argv) {
   const delegations = await hsdProgram.account.delegatedPositionV0.all()
   const needsMigration = delegations.filter(d => d.account.expirationTs.isZero());
   const positionKeys = needsMigration.map(d => d.account.position);
-  const coder = hsdProgram.coder.accounts
+  const coder = vsrProgram.coder.accounts
   const positionAccs = (await getMultipleAccounts({
     connection: provider.connection,
     keys: positionKeys,
-  })).map(a => coder.decode("positionV0", a.data));
+  })).map(a => coder.decode("PositionV0", a.data));
 
   const currTs = await getSolanaUnixTimestamp(provider);
   const currTsBN = new anchor.BN(currTs.toString());
@@ -88,7 +88,7 @@ export async function run(args: any = process.argv) {
           )[0],
           genesisEndSubDaoEpochInfo: subDaoEpochInfoKey(
             subDao,
-            position.genesisEnd
+            position.genesisEnd.isZero() ? min(position.lockup.endTs, proxyEndTs!) : position.genesisEnd
           )[0],
           proxyConfig: registrar.proxyConfig,
           systemProgram: SystemProgram.programId,

programs/helium-sub-daos/src/instructions/delegation/close_delegation_v0.rs

@@ -88,7 +88,14 @@ pub struct CloseDelegationV0<'info> {
         // no need to pass an extra account here. Just pass the closing time sdei and
         // do not change it.
         if position.genesis_end <= registrar.clock_unix_timestamp() {
-          position.lockup.end_ts
+          min(
+              position.lockup.end_ts,
+              if delegated_position.expiration_ts == 0 {
+                position.lockup.end_ts
+              } else {
+                min(position.lockup.end_ts, delegated_position.expiration_ts)
+              }
+            )
         } else {
           position.genesis_end
         }

programs/helium-sub-daos/src/instructions/delegation/delegate_v0.rs

@@ -81,7 +81,7 @@ pub struct DelegateV0<'info> {
         // Avoid passing an extra account if the end is 0 (no genesis on this position).
         // Pass instead closing time epoch info, txn account deduplication will reduce the overall tx size
         if position.genesis_end <= registrar.clock_unix_timestamp() {
-          position.lockup.end_ts
+          min(position.lockup.end_ts, proxy_config.get_current_season(registrar.clock_unix_timestamp()).unwrap().end)
         } else {
           position.genesis_end
         }

programs/helium-sub-daos/src/instructions/delegation/extend_expiration_ts_v0.rs

@@ -81,7 +81,11 @@ pub struct ExtendExpirationTsV0<'info> {
         // no need to pass an extra account here. Just pass the closing time sdei and
         // do not change it.
         if position.genesis_end <= registrar.clock_unix_timestamp() {
-          position.lockup.end_ts
+          if delegated_position.expiration_ts == 0 {
+            position.lockup.end_ts
+          } else {
+            min(position.lockup.end_ts, delegated_position.expiration_ts)
+          }
         } else {
           position.genesis_end
         }