chore: reverse aliasing parameter

Author: albanm

AGENTS.md

@@ -52,7 +52,7 @@ Core: `REGISTRY_URL`, `REGISTRY_SECRET`, `DATA_DIR`, `FONTS_DIR`, `PORT`, `LOG_L
 
 Filtering & aliases (comma-separated):
 - `TILESET_INCLUDE` / `TILESET_EXCLUDE` — whitelist/blacklist tileset IDs
-- `TILESET_ALIASES` — `source:alias` pairs to remap tileset serving keys
+- `TILESET_ALIASES` — `alias:source` pairs to remap tileset serving keys
 - `STYLE_INCLUDE` / `STYLE_EXCLUDE` / `STYLE_ALIASES` — same for styles
 
 ## Testing

src/build-config.ts

@@ -53,7 +53,7 @@ export const buildTileserverConfig = async (): Promise<TileserverConfig> => {
 
   if (config.tilesetInclude.length) log.info(`TILESET_INCLUDE: ${config.tilesetInclude.join(', ')}`)
   if (config.tilesetExclude.length) log.info(`TILESET_EXCLUDE: ${config.tilesetExclude.join(', ')}`)
-  if (Object.keys(config.tilesetAliases).length) log.info(`TILESET_ALIASES: ${Object.entries(config.tilesetAliases).map(([s, a]) => `${s}:${a}`).join(', ')}`)
+  if (Object.keys(config.tilesetAliases).length) log.info(`TILESET_ALIASES: ${Object.entries(config.tilesetAliases).map(([s, a]) => `${a}:${s}`).join(', ')}`)
 
   const filteredTilesets = filterArtefacts(tilesets, config.tilesetInclude, config.tilesetExclude)
   if (filteredTilesets.length !== tilesets.length) {
@@ -79,7 +79,7 @@ export const buildTileserverConfig = async (): Promise<TileserverConfig> => {
 
   if (config.styleInclude.length) log.info(`STYLE_INCLUDE: ${config.styleInclude.join(', ')}`)
   if (config.styleExclude.length) log.info(`STYLE_EXCLUDE: ${config.styleExclude.join(', ')}`)
-  if (Object.keys(config.styleAliases).length) log.info(`STYLE_ALIASES: ${Object.entries(config.styleAliases).map(([s, a]) => `${s}:${a}`).join(', ')}`)
+  if (Object.keys(config.styleAliases).length) log.info(`STYLE_ALIASES: ${Object.entries(config.styleAliases).map(([s, a]) => `${a}:${s}`).join(', ')}`)
 
   const filteredStyles = filterArtefacts(styles, config.styleInclude, config.styleExclude)
   if (filteredStyles.length !== styles.length) {

src/config-parse.ts

@@ -5,8 +5,8 @@ export const parseAliases = (v: string): Record<string, string> => {
   if (!v) return {}
   const map: Record<string, string> = {}
   for (const pair of v.split(',')) {
-    const [source, alias] = pair.split(':').map(s => s.trim())
-    if (source && alias) map[source] = alias
+    const [alias, source] = pair.split(':').map(s => s.trim())
+    if (alias && source) map[source] = alias
   }
   return map
 }

tests/filter-aliases.unit.spec.ts

@@ -26,33 +26,33 @@ test.describe('parseAliases', () => {
     expect(parseAliases('')).toEqual({})
   })
 
-  test('parses source:alias pairs', () => {
-    expect(parseAliases('world:france')).toEqual({ world: 'france' })
+  test('parses alias:source pairs', () => {
+    expect(parseAliases('france:world')).toEqual({ world: 'france' })
   })
 
   test('parses multiple pairs', () => {
-    expect(parseAliases('world:france,contours-2026:contours')).toEqual({
+    expect(parseAliases('france:world,contours:contours-2026')).toEqual({
       world: 'france',
       'contours-2026': 'contours'
     })
   })
 
   test('trims whitespace around keys and values', () => {
-    expect(parseAliases(' world : france , foo : bar ')).toEqual({
+    expect(parseAliases(' france : world , bar : foo ')).toEqual({
       world: 'france',
       foo: 'bar'
     })
   })
 
   test('skips malformed entries without colon', () => {
-    expect(parseAliases('good:pair,nocolon,also:fine')).toEqual({
+    expect(parseAliases('pair:good,nocolon,fine:also')).toEqual({
       good: 'pair',
       also: 'fine'
     })
   })
 
-  test('skips entries with empty source or alias', () => {
-    expect(parseAliases(':alias,source:,ok:ok')).toEqual({ ok: 'ok' })
+  test('skips entries with empty alias or source', () => {
+    expect(parseAliases(':source,alias:,ok:ok')).toEqual({ ok: 'ok' })
   })
 })