Centralize db and partial openapi models packages in a single models package (#317)

* fix bad array element removal

* Project to ProjectName to prevent clash

* suffix for clashing rest and db models

* update gen

* update oapi-codegen

* gen test client server

* update gen - pending Db rename to Models everywhere

* can generate

* fixed gen and tests - pending old models removal

* try update crud gen

* update test client gen

* rest test client with rest types bodies for simpler tests

* bring back rest models for crud gen

* migration ci

* check no type grouping in models.go

* suffix magic file rest models

* refactor client-server gen

* refactor

* schema name guards

* faster camel and pascal and fix spec parameter helper gen

* run go generate separately - current noop

* lint sql dirs concurrently

* memoized to_pascal

* fix crud gen

* update forms

* discriminated ui form

* typed discriminated form default values

* intercept id types in schema arrays

* dont use deprecated swaggest interceptor

* linter update

* cache gen db migrations

* refactor

* update gen

* clean old models

Author: danicc097

.github/workflows/lint-migrations.yml

@@ -0,0 +1,18 @@
+name: Lint Migrations
+
+# on: pull_request
+
+jobs:
+  lint_migrations:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v1
+      - name: Find modified migrations
+        run: |
+          modified_migrations=$(git diff --name-only origin/$GITHUB_BASE_REF...origin/$GITHUB_HEAD_REF 'db/migrations/*.sql')
+          echo "$modified_migrations"
+          echo "::set-output name=file_names::$modified_migrations"
+        id: modified-migrations
+      - uses: sbdchd/squawk-action@v1
+        with:
+          pattern: ${{ steps.modified-migrations.outputs.file_names }}

.golangci.yml

@@ -7,6 +7,7 @@ run:
 linters:
   enable-all: true
   disable:
+    - dupword
     - ireturn # oapi-codegen strict handlers
     # - interfacebloat
     - thelper

Dockerfile.oidc-server

@@ -3,7 +3,7 @@ FROM golang:1.21-alpine3.18 AS build
 WORKDIR /go/src
 COPY ./vendor/ ./vendor/
 COPY ./cmd/oidc-server/main.go go.mod go.sum ./
-COPY ./internal/models/ ./internal/models/
+COPY ./internal/repos/postgresql/gen/models/ ./internal/repos/postgresql/gen/models/
 # RUN go mod download # it will access network/cache, which is not necessary with -mod=vendor
 ENV CGO_ENABLED=0
 RUN --mount=type=cache,target=/root/.cache/go-build \

bin/.helpers.sh

@@ -189,6 +189,22 @@ array.add_suffix() {
   printf "%s\n" "${@/%/$suffix}"
 }
 
+# modifies in place
+array.remove_element() {
+  local -n array=$1
+  local value=$2
+  local temp_array=()
+
+  # Loop through the array and keep only elements that are not equal to the value
+  for element in "${array[@]}"; do
+    if [ "$element" != "$value" ]; then
+      temp_array+=("$element")
+    fi
+  done
+
+  # Assign the modified array back to the original array
+  array=("${temp_array[@]}")
+}
 # breaks when separator has spaces, e.g. " | "
 # join_by() {
 #   [ "$#" -ge 1 ] || return 1
@@ -207,9 +223,20 @@ to_snake() {
   echo "${kebab//-/_}"
 }
 
+# https://stackoverflow.com/questions/57804252/consistent-syntax-for-obtaining-output-of-a-command-efficiently-in-bash
+# also see https://github.com/dimo414/bash-cache if needed for more expensive functions
+declare -Ag memoized_to_pascal
+
+# via nameref
 to_pascal() {
-  local string=$1
-  local pascal_case=""
+  local -n __to_pascal_res="$1"
+  local string="$2"
+
+  local memoized="${memoized_to_pascal[$string]}"
+  if [[ -n "$memoized" ]]; then
+    __to_pascal_res="$memoized"
+    return
+  fi
 
   # Replace spaces with nothing and capitalize the following letter
   string="${string// \([a-z]\)/\U\1}"
@@ -223,18 +250,22 @@ to_pascal() {
 
   for word in $string; do
     if [[ " ${exceptions[*]} " =~ " $word " ]]; then
-      pascal_case+="${word^^}" # Uppercase the whole word
+      __to_pascal_res+="${word^^}" # Uppercase the whole word
     else
-      pascal_case+="${word^}" # Capitalize the first letter
+      __to_pascal_res+="${word^}" # Capitalize the first letter
     fi
   done
 
-  echo "$pascal_case"
+  memoized_to_pascal["$string"]="$__to_pascal_res"
 }
 
+# via nameref
 to_camel() {
-  local pascal_case=$(to_pascal "$1")
-  echo "${pascal_case,}"
+  local -n __to_camel_res="$1"
+  local string="$2"
+
+  to_pascal __to_camel_res "$string"
+  __to_camel_res="${__to_camel_res,}"
 }
 
 function to_kebab() {