* update mu/defn kondo config 2 support lsp renaming

* update mu/defn kondo config 2 support lsp renaming

* add docstrings

* fix linting error

* fix "unused binding" error in test fxn

* fix some calls to mu/defn

[ci skip]

[ci skip]

* Address feedback on #28017

* Fix evil code order

* Pass metadata state from redux

* Filter non-resolved columns

[ci skip]

Co-authored-by: Will Bryant <will.bryant@gmail.com>

* avoid n+1 in search api

* remove the mapping from model to the old toucan model

* use model vars instead of keyword

* Fix shared.util.i18n

* Simplify stuff

* Mind = blown 

* Remove printls

* Simplify things

* Better exampls

* Update dev.clj

* Update Driver Changelog with Honey SQL 2 instructions

* Fix typo

* Add note about renamed functions

* Support Honey SQL 2 for SQL QP

* Sort namespace

* Make Postgres/Redshift be Honey SQL 2-only

* Test fix 

* Fix busted

* Redshift test fixes 

* Sort namespaces

* Fix Postgres convert_timezone

* Fix linter config.

* Fix one last test failure

* Redshift test fix

* Test fixes and some extra validation etc.

* Update common.clj

* Update common.clj

* Update common.clj

* update description

* update api docs with script

* clickhouse from community to partner driver

* format

Adding a dev-level docker-compose file. Located in /dev and is self-documented.

* Add script validating e2e test file names

* Move helper files to helper directories

* Fix whitespace before the first file name

* Add node shebang

Co-authored-by: Nemanja Glumac <31325167+nemanjaglumac@users.noreply.github.com>

* Check if staged files include relevant files from `scenarios` folder

* Distinguish between staged files and all files

* Add precommit hook

* Update binning folder structure

* Improve glob pattern

* Add back forgotten binning constants

* Update cross-version folder structure

* Update embedding folder structure

* Update embedding folder structure

* Update dashboard-filters folder structure

* Use all caps for global vars

* Address feedback: consistency

* Simplify functions

* Actually exit the process

* Adjust script for use with `lint-staged`

* Apply another suggestion to optimize script

* Improve naming of boolean checks

* Fix the init logic

---------

Co-authored-by: Nemanja Glumac <31325167+nemanjaglumac@users.noreply.github.com>

* Better API error message for malli schemas

* add tests

* fix failed test

* fix the backward tests

* add api localized error message test

* add require and fix test

`/api/collection/root/items` now adds `personal_owner_id` to collection results.

[ci skip]

Removing / fixing warnings when updating a dashboard filter and when you have incorrect link templates (#28174)

* Create select-action

* Rename to select-actions

* Remove comments

* Remove unnecessary ()

* Missed one

* Fix GET /action/:uuid

* Use new fns more

* Fix

* Remove select-action-without-implicit-params, we don't need it

* Enable Kondo `:warn-on-reflection` linter

* Enable warnings for metabase.api.setting-test

* Add warn-on-reflection to metabase.util.jvm

Some JVM-specific functions are now in `metabase.util.jvm`, but they
are re-exported (in CLJ) by `metabase.util`.

Make sure we're using a user's full list of permission groups when determining whether a sandbox should be enforced (#28198)

* make sure we're using the full group list when checking whether a sandbox should be enforced

* add simple test that would have caught this bug

https://github.com/camsaul/toucan2/pull/126

Editing the data model loads a _lot_ of data. It hits the following form
from the api:

```clojure
(hydrate [:tables [:fields [:target :has_field_values]
                   :has_field_values] :segments :metrics])
```

On my test instances, with 1500 tables each with 14 fields: 15mb and 359
tables each with 14 fields: 3.69mb. Not great.

If you want to recreate, create an empty db, sync it, then

```clojure
  (time
   (let [conn (sql-jdbc.conn/db->pooled-connection-spec 3)]
     (dotimes [n 359]
       (jdbc/execute! conn [(format "create table if not exists foo_%d (id int, col_1 text, col_2 text, col_3 text, col_4 text, col_5 text, col_6 text, col_7 text, col_8 text, col_9 text, col_10 text, col_11 text, col_12 text, col_13 text, col_14 text)" n)]))))

  (sync/sync-database! (db/select-one 'Database :id 3) {:scan :schema})

  (binding [api/*current-user-permissions-set* (delay #{"/"})]
    (count (#'metabase.api.database/db-metadata 3 true true)))
```

And the source of the bug was a classic recur with a lazy concat.

```diff
diff --git a/src/toucan2/tools/hydrate.clj b/src/toucan2/tools/hydrate.clj
index e1d0459..10c9c49 100644
--- a/src/toucan2/tools/hydrate.clj
+++ b/src/toucan2/tools/hydrate.clj
@@ -456,7 +456,7 @@
     (if (empty? hydrated-instances)
       (concat acc (map :instance annotated-instances))
       (let [[not-hydrated [_needed-hydration & more]] (split-with (complement :needs-hydration?) annotated-instances)]
-        (recur (concat acc (map :instance not-hydrated) [(first hydrated-instances)])
+        (recur (vec (concat acc (map :instance not-hydrated) [(first hydrated-instances)]))
                more
                (rest hydrated-instances))))))

```

* support executing actions in public dashboards

* fix conflict with public actions