fix: sort tariff datapoints newest-first and fix cachedFetchJson test

server/worldmonitor/news/v1/summarize-article.ts

@@ -139,6 +139,7 @@ export async function summarizeArticle(
           .replace(/<reflection>[\s\S]*?<\/reflection>/gi, '')
           .trim();
 
+        // Strip unterminated thinking blocks (no closing tag)
         rawContent = rawContent
           .replace(/<think>[\s\S]*/gi, '')
           .replace(/<\|thinking\|>[\s\S]*/gi, '')

src/components/TradePolicyPanel.ts

@@ -150,7 +150,7 @@ export class TradePolicyPanel extends Panel {
       return `<div class="economic-empty">${t('components.tradePolicy.noTariffData')}</div>`;
     }
 
-    const rows = this.tariffsData.datapoints.map(d =>
+    const rows = [...this.tariffsData.datapoints].sort((a, b) => b.year - a.year).map(d =>
       `<tr>
         <td>${d.year}</td>
         <td>${d.tariffRate.toFixed(1)}%</td>

tests/ttl-acled-ais-guards.test.mjs

@@ -52,13 +52,11 @@ describe('ACLED shared cache layer', () => {
       'Cache key should include event types, start date, end date');
   });
 
-  it('checks Redis cache before upstream API call', () => {
-    const cacheCheckIdx = src.indexOf('getCachedJson(cacheKey)');
-    const fetchIdx = src.indexOf('fetch(`${ACLED_API_URL}');
-    assert.ok(cacheCheckIdx > -1, 'Should check Redis cache');
-    assert.ok(fetchIdx > -1, 'Should call ACLED API');
-    assert.ok(cacheCheckIdx < fetchIdx,
-      'Cache check should come before upstream fetch');
+  it('uses cachedFetchJson to check Redis cache before upstream API call', () => {
+    assert.match(src, /cachedFetchJson\s*<.*>\s*\(cacheKey/,
+      'Should use cachedFetchJson which handles cache check + coalescing');
+    assert.ok(src.includes('fetch(`${ACLED_API_URL}'),
+      'Should call ACLED API inside the fetcher');
   });
 
   it('uses 15-minute cache TTL', () => {
@@ -71,9 +69,9 @@ describe('ACLED shared cache layer', () => {
       'Should gracefully degrade when ACLED_ACCESS_TOKEN is not set');
   });
 
-  it('writes to cache on successful fetch', () => {
-    assert.match(src, /setCachedJson\(cacheKey, events, ACLED_CACHE_TTL\)/,
-      'Should cache successful results');
+  it('caches successful results via cachedFetchJson', () => {
+    assert.match(src, /cachedFetchJson/,
+      'Should use cachedFetchJson which writes to cache automatically on successful fetch');
   });
 
   it('caches empty successful responses to avoid repeated cache misses', () => {