diff --git a/agrifine-extension/src/modules/data-ingest/index.js b/agrifine-extension/src/modules/data-ingest/index.js
index dce5edb..79e4117 100644
--- a/agrifine-extension/src/modules/data-ingest/index.js
+++ b/agrifine-extension/src/modules/data-ingest/index.js
@@ -69,7 +69,10 @@ export function DataIngestModule() {
async _processFile(file, container) {
const status = container.querySelector('#ingest-status');
- const typeName = SUPPORTED_TYPES[file.type] ?? (file.name.endsWith('.csv') ? 'CSV' : null);
+ const typeName = SUPPORTED_TYPES[file.type]
+ ?? (file.name.endsWith('.csv') ? 'CSV'
+ : (file.name.endsWith('.xlsx') || file.name.endsWith('.xls')) ? 'Excel'
+ : null);
if (!typeName) {
status.textContent = 'Unsupported file type.';
diff --git a/agrifine-extension/src/modules/field-profile/index.js b/agrifine-extension/src/modules/field-profile/index.js
index 646f650..958b394 100644
--- a/agrifine-extension/src/modules/field-profile/index.js
+++ b/agrifine-extension/src/modules/field-profile/index.js
@@ -148,7 +148,7 @@ export function FieldProfileModule() {
- ${p.coordinates?.lat ? `
📍 ${p.coordinates.lat.toFixed(4)}, ${p.coordinates.lon.toFixed(4)}
` : ''}
+ ${p.coordinates?.lat != null && p.coordinates?.lon != null ? `
📍 ${p.coordinates.lat.toFixed(4)}, ${p.coordinates.lon.toFixed(4)}
` : ''}
${p.notes ? `
📝 ${p.notes}
` : ''}
Weather data: Phase 6
Carbon potential: Phase 7
diff --git a/agrifine-extension/src/modules/reading-list/index.js b/agrifine-extension/src/modules/reading-list/index.js
index 8c3db7c..47af440 100644
--- a/agrifine-extension/src/modules/reading-list/index.js
+++ b/agrifine-extension/src/modules/reading-list/index.js
@@ -1,6 +1,19 @@
import { getReadingList, saveReadingItem, deleteReadingItem } from '../../utils/storage.js';
import { callAnthropic, AGRICULTURE_TAGS } from '../../utils/api.js';
+function escapeHtml(str) {
+ return String(str ?? '')
+ .replace(/&/g, '&').replace(//g, '>').replace(/"/g, '"');
+}
+
+function safeHref(url) {
+ try {
+ const u = new URL(url);
+ return (u.protocol === 'https:' || u.protocol === 'http:') ? escapeHtml(url) : '#';
+ } catch (_) { return '#'; }
+}
+
export function ReadingListModule() {
let currentTag = 'all';
@@ -126,18 +139,18 @@ export function ReadingListModule() {
}
listEl.innerHTML = filtered.map((item) => `
-
+
- ${item.summary ? `
${item.summary}
` : ''}
+ ${item.summary ? `
${escapeHtml(item.summary)}
` : ''}
- ${(item.tags ?? []).map((t) => `${t}`).join('')}
+ ${(item.tags ?? []).map((t) => `${escapeHtml(t)}`).join('')}
${new Date(item.savedAt).toLocaleDateString()}
diff --git a/agrifine-extension/src/utils/storage.js b/agrifine-extension/src/utils/storage.js
index f5f6b95..680a433 100644
--- a/agrifine-extension/src/utils/storage.js
+++ b/agrifine-extension/src/utils/storage.js
@@ -22,26 +22,38 @@ export const KEYS = {
// ── Generic helpers ──────────────────────────────────────────────────────────
export async function localGet(key) {
- return new Promise((resolve) => {
- chrome.storage.local.get(key, (result) => resolve(result[key] ?? null));
+ return new Promise((resolve, reject) => {
+ chrome.storage.local.get(key, (result) => {
+ if (chrome.runtime.lastError) { reject(new Error(chrome.runtime.lastError.message)); return; }
+ resolve(result[key] ?? null);
+ });
});
}
export async function localSet(key, value) {
- return new Promise((resolve) => {
- chrome.storage.local.set({ [key]: value }, resolve);
+ return new Promise((resolve, reject) => {
+ chrome.storage.local.set({ [key]: value }, () => {
+ if (chrome.runtime.lastError) { reject(new Error(chrome.runtime.lastError.message)); return; }
+ resolve();
+ });
});
}
export async function sessionGet(key) {
- return new Promise((resolve) => {
- chrome.storage.session.get(key, (result) => resolve(result[key] ?? null));
+ return new Promise((resolve, reject) => {
+ chrome.storage.session.get(key, (result) => {
+ if (chrome.runtime.lastError) { reject(new Error(chrome.runtime.lastError.message)); return; }
+ resolve(result[key] ?? null);
+ });
});
}
export async function sessionSet(key, value) {
- return new Promise((resolve) => {
- chrome.storage.session.set({ [key]: value }, resolve);
+ return new Promise((resolve, reject) => {
+ chrome.storage.session.set({ [key]: value }, () => {
+ if (chrome.runtime.lastError) { reject(new Error(chrome.runtime.lastError.message)); return; }
+ resolve();
+ });
});
}