feat: refactor the search API to use query params for the filters instead of "natural language"

.github/workflows/autofix.yml

@@ -61,25 +61,27 @@ jobs:
       - name: Setup | Checkout
         uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
         with: {persist-credentials: false}
+      - name: Setup | Pull Docker images
+        run: docker compose pull
+      - name: Setup | Build Docker images
+        run: docker compose -f compose.local.yml build server
+
       - name: Setup | pnpm
         uses: pnpm/action-setup@08c4be7e2e672a47d11bd04269e27e5f3e8529cb # v6.0.0
         with:
-          version: latest
-
-      - name: Setup | Node.js
-        uses: actions/setup-node@6044e13b5dc448c55e2357c09f80417699197238 # v6.2.0
-        with:
-          cache: 'pnpm'
-          cache-dependency-path: tests/pnpm-lock.yaml
-
-      - name: Install | Dependencies
-        run: pnpm install --frozen-lockfile
-        working-directory: tests
+          version: 10.14.0
+          package_json_file: tests/package.json
+          run_install: |
+            - cwd: tests
+              args: [--frozen-lockfile]
 
       - name: Update | Screenshots
         run: pnpm run update-screenshots
         working-directory: tests
 
+      - name: Restore | Discard lockfile side-effects from pnpm
+        run: git checkout -- tests/pnpm-lock.yaml
+
       - uses: autofix-ci/action@635ffb0c9798bd160680f18fd73371e355b85f27 # v1.3.2
         with:
           commit-message: "chore: update screenshots"
@@ -121,17 +123,11 @@ jobs:
       - name: Setup | pnpm
         uses: pnpm/action-setup@08c4be7e2e672a47d11bd04269e27e5f3e8529cb # v6.0.0
         with:
-          version: latest
-
-      - name: Setup | Node.js
-        uses: actions/setup-node@6044e13b5dc448c55e2357c09f80417699197238 # v6.2.0
-        with:
-          cache: 'pnpm'
-          cache-dependency-path: webclient/pnpm-lock.yaml
-
-      - name: Install | Dependencies
-        run: pnpm install --frozen-lockfile
-        working-directory: webclient
+          version: 10.14.0
+          package_json_file: webclient/package.json
+          run_install: |
+            - cwd: webclient
+              args: [--frozen-lockfile]
 
       - name: Build | Compile server
         working-directory: server
@@ -183,6 +179,9 @@ jobs:
         run: pnpm run type-refresh
         working-directory: webclient
 
+      - name: Restore | Discard lockfile side-effects from pnpm
+        run: git checkout -- webclient/pnpm-lock.yaml
+
       - uses: autofix-ci/action@635ffb0c9798bd160680f18fd73371e355b85f27 # v1.3.2
         with:
           commit-message: "chore: Update OpenAPI spec and types"

Cargo.toml

@@ -42,6 +42,7 @@ rustls = "0.23.36"
 serde = { version = "1.0", features = ["derive"] }
 serde_json = { version = "1.0.149", features = ["raw_value"] }
 serde_with = { version = "3.16.1", default-features = false, features = [] }
+serde_html_form = "0.4.0"
 serde_yaml = "0.9.34"
 sqlx = { version = "0.8.6", features = ['chrono', 'json', 'macros', 'migrate', 'postgres', 'runtime-tokio', 'tls-rustls'], default-features = false }
 tempfile = "3.25.0"

data/output/openapi.yaml

@@ -570,16 +570,50 @@ paths:
           description: |-
             string you want to search for.
 
-            The amounts returned can be controlled using the `limit\*` paramerters.
-
-            The following query-filters are supported:
-            - `in:<parent>`/`@<parent>`: Only return rooms in the given parent (e.g. `in:5304` or `in:garching`)
-            - `usage:<type>`/`nutzung:<usage>`/`=<usage>`: Only return entries of the given usage (e.g. `usage:wc` or `usage:büro`)
-            - `type:<type>`: Only return entries of the given type (e.g. `type:building` or `type:room`)
-            - `near:<lat>,<lon>`: prioritise sorting the entries by distance to a coordinate
+            The amounts returned can be controlled using the `limit_*` parameters.
+            Use `in`, `usage`, `type`, and `near` query parameters for filtering.
           required: true
           schema:
             type: string
+        - name: in
+          in: query
+          description: |-
+            Filter by parent (building, campus, etc.).
+
+            Can be repeated for multiple values (e.g. `&in=garching&in=5304`).
+          required: false
+          schema:
+            type: array
+            items:
+              type: string
+        - name: usage
+          in: query
+          description: |-
+            Filter by usage type (e.g. `wc`, `büro`).
+
+            Can be repeated for multiple values.
+          required: false
+          schema:
+            type: array
+            items:
+              type: string
+        - name: type
+          in: query
+          description: |-
+            Filter by entry type (e.g. `room`, `building`).
+
+            Can be repeated for multiple values.
+          required: false
+          schema:
+            type: array
+            items:
+              type: string
+        - name: near
+          in: query
+          description: Sort results by distance to a coordinate (`lat,lon`).
+          required: false
+          schema:
+            type: string
         - name: search_addresses
           in: query
           description: |-
@@ -589,9 +623,7 @@ paths:
             Only activate this when you really need it.
           required: false
           schema:
-            type:
-              - boolean
-              - "null"
+            type: boolean
         - name: limit_buildings
           in: query
           description: |-
@@ -601,9 +633,7 @@ paths:
             If this is a problem for you, please open an issue.
           required: false
           schema:
-            type:
-              - integer
-              - "null"
+            type: integer
             minimum: 0
         - name: limit_rooms
           in: query
@@ -614,9 +644,7 @@ paths:
             If this is an problem for you, please open an issue.
           required: false
           schema:
-            type:
-              - integer
-              - "null"
+            type: integer
             minimum: 0
         - name: limit_all
           in: query
@@ -627,9 +655,7 @@ paths:
             If this is an problem for you, please open an issue.
           required: false
           schema:
-            type:
-              - integer
-              - "null"
+            type: integer
             minimum: 0
         - name: pre_highlight
           in: query
@@ -640,9 +666,7 @@ paths:
             For background on the default values, please see [Wikipedia](https://en.wikipedia.org/wiki/C0_and_C1_control_codes#Modified_C0_control_code_sets)).
           required: false
           schema:
-            type:
-              - string
-              - "null"
+            type: string
         - name: post_highlight
           in: query
           description: |-
@@ -652,9 +676,7 @@ paths:
             For background on the default values, please see [Wikipedia](https://en.wikipedia.org/wiki/C0_and_C1_control_codes#Modified_C0_control_code_sets)).
           required: false
           schema:
-            type:
-              - string
-              - "null"
+            type: string
         - name: cropping
           in: query
           description: |-

server/Cargo.toml

@@ -28,6 +28,7 @@ anyhow = { workspace=true, features = ["backtrace"] }
 serde = { workspace=true, features = ["derive"] }
 serde_with = { workspace=true, default-features = false, features = [] }
 serde_json = { workspace=true, features = ["raw_value"] }
+serde_html_form.workspace=true
 serde_yaml.workspace=true
 
 # runtime + webserver