@emeryld/rrroutes-export 1.0.26 → 1.0.27

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@emeryld/rrroutes-export",
   "description": "Finalized leaves export helpers and CLI for RRRoutes",
-  "version": "1.0.26",
+  "version": "1.0.27",
   "private": false,
   "type": "module",
   "main": "dist/index.cjs",

package/tools/finalized-leaves-viewer.html

@@ -423,24 +423,6 @@
       .changelog-schema-type {
         color: var(--schema-accent);
       }
-      .changelog-schema-details {
-        margin-top: 6px;
-      }
-      .changelog-schema-details > summary {
-        color: var(--schema-accent);
-        font-size: 12px;
-      }
-      .changelog-schema-details-body {
-        margin-top: 6px;
-        display: grid;
-        gap: 4px;
-      }
-      .changelog-schema-line > summary {
-        font-size: 12px;
-      }
-      .changelog-schema-line .meta {
-        margin-top: 4px;
-      }
 
       .event-title-row {
         display: inline-flex;
@@ -696,14 +678,6 @@
               >
                 Changelog
               </button>
-              <button
-                id="tabChangelogGrouped"
-                class="field-chip"
-                type="button"
-                aria-pressed="false"
-              >
-                Grouped Changelog
-              </button>
             </div>
           </div>
         </div>
@@ -892,7 +866,6 @@
       const VIEW_IDS = {
         leaves: 'leaves',
         changelog: 'changelog',
-        changelogGrouped: 'changelogGrouped',
       }
 
       const VIEWER_MODE = {
@@ -927,12 +900,6 @@
             CHANGELOG_TIMELINE_FIELDS.map((field) => field.id),
           ),
         },
-        [VIEW_IDS.changelogGrouped]: {
-          fieldDefs: CHANGELOG_TIMELINE_FIELDS,
-          defaultSelected: new Set(
-            CHANGELOG_TIMELINE_FIELDS.map((field) => field.id),
-          ),
-        },
       }
 
       const state = {
@@ -950,9 +917,6 @@
           [VIEW_IDS.changelog]: new Set(
             TAB_CONFIG[VIEW_IDS.changelog].defaultSelected,
           ),
-          [VIEW_IDS.changelogGrouped]: new Set(
-            TAB_CONFIG[VIEW_IDS.changelogGrouped].defaultSelected,
-          ),
         },
       }
 
@@ -978,9 +942,6 @@
       const controlsEl = document.querySelector('.controls')
       const tabLeavesBtn = document.getElementById('tabLeaves')
       const tabChangelogBtn = document.getElementById('tabChangelog')
-      const tabChangelogGroupedBtn = document.getElementById(
-        'tabChangelogGrouped',
-      )
 
       const URL_PARAM_KEY = 'filters'
       let isHydratingFromUrl = false
@@ -1718,8 +1679,9 @@
           content.appendChild(separatedSchemas)
         }
 
-        const routeHistory = state.changelogPayload?.byRoute?.[leaf.key]
-        if (Array.isArray(routeHistory) && routeHistory.length > 0) {
+        const routeHistoryRaw = state.changelogPayload?.byRoute?.[leaf.key]
+        const routeHistory = Array.isArray(routeHistoryRaw) ? routeHistoryRaw : []
+        if (state.changelogPayload) {
           const commitBySha = new Map(
             (state.changelogPayload?.commits || []).map((commit) => [
               commit.sha,
@@ -1744,20 +1706,24 @@
             '',
             `Route Changelog (${routeHistory.length} event${routeHistory.length === 1 ? '' : 's'})`,
           )
-          if (state.viewerMode === VIEWER_MODE.bundle) {
-            routeChangelogSummary.classList.add('route-changelog-summary-gray')
-          }
+          routeChangelogSummary.classList.add('route-changelog-summary-gray')
           routeChangelogDetails.appendChild(routeChangelogSummary)
 
           const routeChangelogBody = el('div', 'leaf-content')
-          routeHistoryNewestFirst.forEach((event) => {
+          if (routeHistoryNewestFirst.length === 0) {
             routeChangelogBody.appendChild(
-              renderEventCard(event, commitBySha, {
-                showTitle: true,
-                item: { eventKind: 'route', routeKey: leaf.key, event },
-              }),
+              el('div', 'meta', 'No changelog entries for this route.'),
             )
-          })
+          } else {
+            routeHistoryNewestFirst.forEach((event) => {
+              routeChangelogBody.appendChild(
+                renderEventCard(event, commitBySha, {
+                  showTitle: true,
+                  item: { eventKind: 'route', routeKey: leaf.key, event },
+                }),
+              )
+            })
+          }
           routeChangelogDetails.appendChild(routeChangelogBody)
           routeChangelogSection.appendChild(routeChangelogDetails)
           content.appendChild(routeChangelogSection)
@@ -2147,54 +2113,6 @@
         return row
       }
 
-      function schemaDiffRawDetail(delta) {
-        if (!delta || typeof delta !== 'object') return ''
-        if ('path' in delta) {
-          const location = formatPathLabel(delta.path) || String(delta.path || '')
-          if (delta.op === 'added') {
-            return `added ${location}: ${formatRouteSchemaEntry(delta.after)}`
-          }
-          if (delta.op === 'removed') {
-            return `removed ${location}: ${formatRouteSchemaEntry(delta.before)}`
-          }
-          return `changed ${location}: ${formatRouteSchemaEntry(delta.before)} -> ${formatRouteSchemaEntry(delta.after)}`
-        }
-        if ('section' in delta) {
-          const location = sourceSchemaLocation(delta.section, delta.path)
-          if (delta.op === 'added') {
-            return `added ${location}: ${JSON.stringify(delta.after || [])}`
-          }
-          if (delta.op === 'removed') {
-            return `removed ${location}: ${JSON.stringify(delta.before || [])}`
-          }
-          return `changed ${location}: ${JSON.stringify(delta.before || [])} -> ${JSON.stringify(delta.after || [])}`
-        }
-        return JSON.stringify(delta)
-      }
-
-      function renderSchemaDiffNested(schemaDiff) {
-        if (!Array.isArray(schemaDiff) || schemaDiff.length === 0) return null
-        const wrapper = el('details', 'changelog-schema-details')
-        wrapper.appendChild(
-          el('summary', 'mono', `schema changes (${schemaDiff.length})`),
-        )
-        const body = el('div', 'changelog-schema-details-body')
-        schemaDiff.forEach((delta) => {
-          const lineNode = formatSchemaDiffLine(delta)
-          if (!lineNode) return
-          const lineDetails = el('details', 'changelog-schema-line')
-          const lineSummary = el('summary')
-          lineSummary.appendChild(lineNode)
-          lineDetails.appendChild(lineSummary)
-          lineDetails.appendChild(
-            el('div', 'meta mono', schemaDiffRawDetail(delta)),
-          )
-          body.appendChild(lineDetails)
-        })
-        wrapper.appendChild(body)
-        return wrapper
-      }
-
       function renderEventCard(event, commitBySha, options = {}) {
         const card = el('div', 'leaf-content')
         const commit = commitBySha.get(event.commitSha)
@@ -2212,8 +2130,16 @@
           card.appendChild(renderEventTitle(options.item))
         }
         if (Array.isArray(event.schemaDiff) && event.schemaDiff.length > 0) {
-          const schemaDetails = renderSchemaDiffNested(event.schemaDiff)
-          if (schemaDetails) card.appendChild(schemaDetails)
+          card.appendChild(
+            el('div', 'meta', `schema changes: ${event.schemaDiff.length}`),
+          )
+          const schemaChanges = el('div')
+          event.schemaDiff.forEach((delta) => {
+            const lineNode = formatSchemaDiffLine(delta)
+            if (!lineNode) return
+            schemaChanges.appendChild(lineNode)
+          })
+          card.appendChild(schemaChanges)
         }
         if (Array.isArray(event.cfgDiff) && event.cfgDiff.length > 0) {
           card.appendChild(
@@ -2310,8 +2236,11 @@
         row.appendChild(renderEventTitle(item))
 
         if (Array.isArray(event.schemaDiff) && event.schemaDiff.length > 0) {
-          const schemaDetails = renderSchemaDiffNested(event.schemaDiff)
-          if (schemaDetails) row.appendChild(schemaDetails)
+          event.schemaDiff.forEach((delta) => {
+            const lineNode = formatSchemaDiffLine(delta)
+            if (!lineNode) return
+            row.appendChild(lineNode)
+          })
         }
         if (Array.isArray(event.cfgDiff) && event.cfgDiff.length > 0) {
           event.cfgDiff.forEach((diff) => {
@@ -2529,10 +2458,7 @@
       function changelogItemsByTab(tabId) {
         const payload = state.changelogPayload
         if (!payload) return []
-        if (
-          tabId === VIEW_IDS.changelog ||
-          tabId === VIEW_IDS.changelogGrouped
-        ) {
+        if (tabId === VIEW_IDS.changelog) {
           return buildChronologicalEvents(payload)
         }
         return []
@@ -2561,10 +2487,6 @@
       function renderTabButtons() {
         setPressed(tabLeavesBtn, state.activeTab === VIEW_IDS.leaves)
         setPressed(tabChangelogBtn, state.activeTab === VIEW_IDS.changelog)
-        setPressed(
-          tabChangelogGroupedBtn,
-          state.activeTab === VIEW_IDS.changelogGrouped,
-        )
         tabLeavesBtn.style.display = state.availableTabs.has(VIEW_IDS.leaves)
           ? ''
           : 'none'
@@ -2572,10 +2494,6 @@
           VIEW_IDS.changelog,
         )
           ? ''
-          : 'none'
-        tabChangelogGroupedBtn.style.display = state.availableTabs.has(
-          VIEW_IDS.changelogGrouped,
-        )
           ? ''
           : 'none'
         const isLeavesTab = state.activeTab === VIEW_IDS.leaves
@@ -2632,24 +2550,10 @@
           )
           if (filtered.length === 0) {
             resultsEl.appendChild(el('div', 'empty', 'No matches.'))
-          } else if (state.activeTab === VIEW_IDS.changelog) {
+          } else {
             renderChangelogWithCommitSeparators(filtered, commitBySha, (item) =>
               renderChangelogEventLines(item, commitBySha),
             )
-          } else {
-            renderChangelogWithCommitSeparators(filtered, commitBySha, (item) => {
-              const details = el('details', 'leaf')
-              const summary = el('summary')
-              summary.appendChild(renderEventTitle(item))
-              details.appendChild(summary)
-              details.appendChild(
-                renderEventCard(item.event, commitBySha, {
-                  showTitle: false,
-                  item,
-                }),
-              )
-              return details
-            })
           }
         }
         syncFilterStateToUrl()
@@ -2674,11 +2578,7 @@
             : [
                 {
                   id: 'tabFields',
-                  label:
-                    state.activeTab === VIEW_IDS.changelog ||
-                    state.activeTab === VIEW_IDS.changelogGrouped
-                      ? 'Changelog fields'
-                      : 'Source fields',
+                  label: 'Changelog fields',
                   fieldIds: fieldDefs.map((field) => field.id),
                 },
               ]
@@ -2864,8 +2764,7 @@
           isHydratingFromUrl = true
           if (
             parsed.activeTab === VIEW_IDS.leaves ||
-            parsed.activeTab === VIEW_IDS.changelog ||
-            parsed.activeTab === VIEW_IDS.changelogGrouped
+            parsed.activeTab === VIEW_IDS.changelog
           ) {
             state.activeTab = parsed.activeTab
           }
@@ -2954,18 +2853,13 @@
         }
         if (state.changelogPayload) {
           state.availableTabs.add(VIEW_IDS.changelog)
-          if (state.viewerMode === VIEWER_MODE.bundle) {
-            state.availableTabs.add(VIEW_IDS.changelogGrouped)
-          }
         }
 
         if (!state.availableTabs.has(state.activeTab)) {
           if (state.availableTabs.has(VIEW_IDS.leaves)) {
             state.activeTab = VIEW_IDS.leaves
-          } else if (state.availableTabs.has(VIEW_IDS.changelog)) {
-            state.activeTab = VIEW_IDS.changelog
           } else {
-            state.activeTab = VIEW_IDS.changelogGrouped
+            state.activeTab = VIEW_IDS.changelog
           }
         }
 
@@ -3029,9 +2923,6 @@
       tabChangelogBtn.addEventListener('click', () =>
         setActiveTab(VIEW_IDS.changelog),
       )
-      tabChangelogGroupedBtn.addEventListener('click', () =>
-        setActiveTab(VIEW_IDS.changelogGrouped),
-      )
 
       selectAllFieldsBtn.addEventListener('click', () =>
         setFieldSelection(new Set(currentFieldDefs().map((field) => field.id))),