fix: persist DA included height in syncer for full nodes

Full nodes syncing from DA were not persisting the DAIncludedHeightKey metadata, causing the E2E test to fail when querying this value. The submitter (sequencer) already persists this key when verifying DA inclusion. This change adds the same logic to the syncer so full nodes also track which blocks have been DA included.
randygrok · Nov 27, 2025 · Nov 28, 2025 · Nov 28, 2025 · Nov 28, 2025 · Nov 28, 2025
commit 3c0a15c4e7bc9334d05d6503b6bcc49596a10ee3
diff --git a/block/internal/syncing/syncer.go b/block/internal/syncing/syncer.go
@@ -464,6 +464,13 @@ func (s *Syncer) processHeightEvent(event *common.DAHeightEvent) {
 
 	// only save to p2p stores if the event came from DA
 	if event.Source == common.SourceDA {
+		// Persist DA included height for full nodes syncing from DA
+		bz := make([]byte, 8)
+		binary.LittleEndian.PutUint64(bz, height)
+		if err := s.store.SetMetadata(s.ctx, store.DAIncludedHeightKey, bz); err != nil {
+			s.logger.Error().Err(err).Uint64("height", height).Msg("failed to persist DA included height")
+		}
+
 		g, ctx := errgroup.WithContext(s.ctx)
 		g.Go(func() error {
 			// broadcast header locally only — prevents spamming the p2p network with old height notifications,