fix(import): reject empty Gemini Takeout prompt blocks

rodboev · rodboev · commit 3109b2c3a4db · 2026-08-01T12:54:05.000-04:00
diff --git a/internal/parser/gemini_apps_takeout.go b/internal/parser/gemini_apps_takeout.go
@@ -526,8 +526,11 @@ func splitGeminiAppsBlocks(tokens []html.Token) []string {
 	var block []html.Token
 	blockDepth := 0
 	flush := func() {
+		if block == nil {
+			return
+		}
 		value := strings.TrimSpace(renderGeminiAppsTokens(block))
-		if value != "" && !geminiAppsTimestampRE.MatchString(value) {
+		if !geminiAppsTimestampRE.MatchString(value) {
 			blocks = append(blocks, value)
 		}
 		block = nil
diff --git a/internal/parser/gemini_apps_takeout_test.go b/internal/parser/gemini_apps_takeout_test.go
@@ -143,6 +143,26 @@ func TestParseGeminiAppsBrStaysInsidePromptBlock(t *testing.T) {
 	assert.Equal(t, "answer", results[0].Messages[1].Content)
 }
 
+func TestParseGeminiAppsEmptyFirstContentBlockIsError(t *testing.T) {
+	fixture := `<!doctype html><html><head><title>My Activity History</title></head><body>
+<div class="outer-cell"><div class="header-cell"><h3>Gemini Apps</h3><p>Prompted</p><p>Jan 2, 2025, 3:04:05 PM EDT</p></div><div class="content-cell"><p></p><p>answer</p></div></div>
+</body></html>`
+	path := filepath.Join(t.TempDir(), "empty-first-block.html")
+	require.NoError(t, os.WriteFile(path, []byte(fixture), 0o644))
+
+	provider, ok := NewProvider(AgentGeminiApps, ProviderConfig{})
+	require.True(t, ok)
+	exporter := provider.(GeminiAppsExportParser)
+	var results []ParseResult
+	summary, err := exporter.ParseGeminiAppsExport(path, func(result ParseResult) error {
+		results = append(results, result)
+		return nil
+	})
+	assert.ErrorContains(t, err, "no admissible Prompted records")
+	assert.Empty(t, results)
+	assert.Equal(t, 1, summary.Errors)
+}
+
 func TestParseGeminiAppsTimestampMustBeInHeader(t *testing.T) {
 	fixture := strings.Replace(
 		sanitizedGeminiAppsHTML,
diff --git a/internal/parser/import_only_provider_test.go b/internal/parser/import_only_provider_test.go
@@ -12,11 +12,13 @@ func TestImportOnlyProviderExportCapabilitiesAreAgentSpecific(t *testing.T) {
 	require.True(t, ok)
 	assert.Implements(t, (*ChatGPTExportParser)(nil), chatGPTProvider)
 	assert.NotImplements(t, (*ClaudeAIExportParser)(nil), chatGPTProvider)
+	assert.NotImplements(t, (*GeminiAppsExportParser)(nil), chatGPTProvider)
 
 	claudeAIProvider, ok := NewProvider(AgentClaudeAI, ProviderConfig{})
 	require.True(t, ok)
 	assert.Implements(t, (*ClaudeAIExportParser)(nil), claudeAIProvider)
 	assert.NotImplements(t, (*ChatGPTExportParser)(nil), claudeAIProvider)
+	assert.NotImplements(t, (*GeminiAppsExportParser)(nil), claudeAIProvider)
 
 	geminiAppsProvider, ok := NewProvider(AgentGeminiApps, ProviderConfig{})
 	require.True(t, ok)
