feat/modules #45

Merged

yorunikakeru merged 6 commits from feat/modules into main 2026-06-21 14:45:07 +02:00

Conversation 0 Commits 6 Files changed 16 +300 -256

yorunikakeru commented 2026-06-21 14:27:34 +02:00

Owner

Copy link

No description provided.

yorunikakeru added 5 commits 2026-06-21 14:27:34 +02:00

refactor(module): extract toWord16 and hardcodedModuleName to Module.Internal ... 627f943b2e

Removes three copies of `toWord16 :: Int -> Word16` and three copies of
the `either (error "impossible") id (mkModuleName "...")` pattern, each
previously duplicated across Forgejo, Nginx, and PostgreSQL builders.

refactor(port): replace Num PortSpec with explicit *WithFallback functions ... 869bbd93e5

Removes IsPort typeclass, Num PortSpec instance, and withFallback.
All port builder functions now take Int for the primary port; fallback
variants (httpPortWithFallback, httpsPortWithFallback, sshPortWithFallback,
portWithFallback) take Int -> [Int] -> builder (), making the fallback
intent explicit and eliminating the silent data loss on PortSpec arithmetic.

refactor(port): replace *WithFallback functions with port + withFallback combinators ... 839a53c0b6

All builder functions take PortSpec again. Port exposes two clean constructors:
  port :: Int -> PortSpec          -- simple case
  withFallback :: Int -> [Int] -> PortSpec  -- with fallbacks

Usage: httpPort (port 80), httpsPort (443 `withFallback` [80]).
Removes the per-module *WithFallback duplicates added in the previous commit.

chore(stan): suppress STAN-0206 via .stan.toml; drop --no-default from AGENTS.md ... 68ecff21f6

STAN-0206 (non-strict fields) is a false positive: StrictData is already
enabled globally for the library via default-extensions in DSL.cabal.
Exclude the inspection project-wide rather than adding redundant ! annotations.

refactor(nginx): move vhost policies from module to virtual host level ... db6a37d0bc

`derivePolicies` lived on `NginxState`, aggregating across all vhosts.
Policies now derived per `NginxVHostState` and stored in `vhostPolicies`;
`nginxPolicies` is always empty.

Also extracts `validatePortSpec` to eliminate the duplicated port
validation block shared by `httpPort` and `httpsPort`.

yorunikakeru added 1 commit 2026-06-21 14:44:17 +02:00

refactor(module): replace hardcodedModuleName with moduleDomainName ... b17b2b92b6

Module builders now derive names via Domain.Module.moduleDomainName
instead of repeating a partial Text → ModuleName conversion locally.
toWord16 also removed in favour of inline fromIntegral.

docs: update README for module architecture — services → modules,
Service/ServiceBuilder → Module/ConfigBuilder, IR JSON version 6,
add module builder examples and package declarations.

yorunikakeru merged commit 22f21672ce into main 2026-06-21 14:45:07 +02:00

yorunikakeru referenced this pull request from a commit 2026-06-21 14:45:08 +02:00

Merge pull request 'feat/modules' (#45) from feat/modules into main

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

FrogOS/DSL!45

No description provided.