wdevs/relspecgo
5
0
Fork 0
Code Issues Pull Requests Actions Packages Releases 82 Wiki Activity

Job manifests cannot preserve Bun --types sqltypes and templ text outputs #29

New Issue
Closed
opened 2026-09-20 20:34:14 +00:00 by warkanum · 4 comments
warkanum commented 2026-09-20 20:34:14 +00:00
Owner
Copy Link
Copy Source

Context

Origin is adopting relspec.yml jobs to replace repeated Makefile RelSpec invocations.

Problem

The job manifest options schema does not expose the convert writer's --types option. In particular, a repository that currently runs:

relspec convert --from dbml --from-list "..." --to bun --to-path internal/models/ --package models --types sqltypes

cannot represent the required sqltypes setting in a job. Omitting it changes generated Bun model types.

Also, templ jobs write rendered text (for example a Go source file), but the manifest requires an output.format from the database writer format list. This makes it unclear or impossible to declare a templ job targeting a .go file without using an unrelated format such as pgsql.

Requested change

  1. Add a job option equivalent to convert --types (and related Bun/GORM writer options such as array-nullable and extra-fields if intended to be supported).
  2. Define a valid output representation for templ jobs, or allow a text output path without requiring a database format.
  3. Add example job manifests/tests covering Bun sqltypes output and database-mode templating to prevent regressions.

Observed with RelSpec v1.0.82 (built 2026-09-20).

## Context Origin is adopting `relspec.yml` jobs to replace repeated Makefile RelSpec invocations. ## Problem The job manifest `options` schema does not expose the `convert` writer's `--types` option. In particular, a repository that currently runs: ```sh relspec convert --from dbml --from-list "..." --to bun --to-path internal/models/ --package models --types sqltypes ``` cannot represent the required `sqltypes` setting in a job. Omitting it changes generated Bun model types. Also, `templ` jobs write rendered text (for example a Go source file), but the manifest requires an `output.format` from the database writer format list. This makes it unclear or impossible to declare a templ job targeting a `.go` file without using an unrelated format such as `pgsql`. ## Requested change 1. Add a job option equivalent to `convert --types` (and related Bun/GORM writer options such as `array-nullable` and `extra-fields` if intended to be supported). 2. Define a valid output representation for `templ` jobs, or allow a text output path without requiring a database format. 3. Add example job manifests/tests covering Bun `sqltypes` output and database-mode templating to prevent regressions. Observed with RelSpec v1.0.82 (built 2026-09-20).
warkanum commented 2026-09-20 20:36:01 +00:00
Author
Owner
Copy Link
Copy Source

Check for more gaps in the options for different writers/readers and add those to the job file implementation

Check for more gaps in the options for different writers/readers and add those to the job file implementation
sgcommand commented 2026-09-20 21:05:47 +00:00
Member
Copy Link
Copy Source

Dispatcher assignment — 2026-09-20T23:05:22+02:00

  • Forge/repository: Gitea wdevs/relspecgo
  • Issue: #29 — Job manifests cannot preserve Bun --types sqltypes and templ text outputs
  • Classification: non-simple Go manifest/schema implementation requiring design judgment; not eligible for Ornith.
  • Selected backend: Hermes OpenAI/Codex surface, model gpt-5.6-luna.
  • Fallback reason: Claude Code CLI live checks passed (claude 2.1.278, authenticated), but bounded Sonnet probe failed with the account monthly spend-limit message. A bounded gpt-5.6-luna probe returned CODEX_56_OK.
  • Worker session/process: ia-relspecgo-29-gpt56 / pending launch after this comment.
  • Worktree: /home/hermes/work/issue-agents/wdevs/relspecgo/issue-29-job-manifest-options
  • Branch: issue-29-job-manifest-options (created locally from synchronized master; not pushed yet)
  • PR: not created yet.
  • Taskboard: card t_4364444d created/read back as blocked to suppress duplicate board dispatch.
  • Plan: read the issue and repository CLAUDE.md (no root AGENTS.md exists); inspect job manifest schemas, readers/writers, and existing tests; implement focused options/text-output support with regression coverage; run format/tests/build; commit, push, create a non-empty PR against master, and post exact SHA/PR/verification evidence.
  • Required worker skills: cron-model-selection-policy, cron-worker-contract.
## Dispatcher assignment — 2026-09-20T23:05:22+02:00 - **Forge/repository:** Gitea `wdevs/relspecgo` - **Issue:** #29 — Job manifests cannot preserve Bun `--types sqltypes` and templ text outputs - **Classification:** non-simple Go manifest/schema implementation requiring design judgment; not eligible for Ornith. - **Selected backend:** Hermes OpenAI/Codex surface, model `gpt-5.6-luna`. - **Fallback reason:** Claude Code CLI live checks passed (`claude` 2.1.278, authenticated), but bounded Sonnet probe failed with the account monthly spend-limit message. A bounded `gpt-5.6-luna` probe returned `CODEX_56_OK`. - **Worker session/process:** `ia-relspecgo-29-gpt56` / pending launch after this comment. - **Worktree:** `/home/hermes/work/issue-agents/wdevs/relspecgo/issue-29-job-manifest-options` - **Branch:** `issue-29-job-manifest-options` (created locally from synchronized `master`; not pushed yet) - **PR:** not created yet. - **Taskboard:** card `t_4364444d` created/read back as blocked to suppress duplicate board dispatch. - **Plan:** read the issue and repository `CLAUDE.md` (no root `AGENTS.md` exists); inspect job manifest schemas, readers/writers, and existing tests; implement focused options/text-output support with regression coverage; run format/tests/build; commit, push, create a non-empty PR against `master`, and post exact SHA/PR/verification evidence. - **Required worker skills:** `cron-model-selection-policy`, `cron-worker-contract`.
sgcommand commented 2026-09-20 21:18:26 +00:00
Member
Copy Link
Copy Source

Implemented in PR #30: #30

Branch: issue-29-job-manifest-options
Commit: ade71b75fa

Summary:

  • Job manifest options.types and options.array_nullable now reach the existing Bun/GORM writer options for convert/merge/split output paths.
  • templ accepts the explicit templ-only output.format: text marker for arbitrary text output; omitted format remains compatible.
  • Added regression tests for Bun sqltypes preservation and templ text validation/execution, plus documentation/example updates.

Changed paths:

  • pkg/jobs/jobs.go
  • cmd/relspec/job.go
  • cmd/relspec/job_test.go
  • docs/JOB_FILES.md
  • examples/jobs/relspec.reports.yml

Verification:

  • go test ./pkg/jobs ./cmd/relspec: PASS
  • go build -o /home/hermes/.tmp/relspec-issue-29 ./cmd/relspec: PASS
  • go vet ./...: PASS
  • git diff --check: PASS
  • go test ./...: BLOCKED by pre-existing environment failure in pkg/writers/dctx TestWriter_WriteSchema: open /tmp/test.dctx: permission denied
  • make lint: BLOCKED because installed golangci-lint v1 rejects repository golangci-lint v2 configuration

Model: gpt-5.6-luna via Hermes openai-codex fallback because Claude spend limit.
Blockers: full suite and lint blockers above; no implementation blocker.

Implemented in PR #30: https://git.warky.dev/wdevs/relspecgo/pulls/30 Branch: issue-29-job-manifest-options Commit: ade71b75fa5a4b5e37e3e824ae027163348be1d9 Summary: - Job manifest options.types and options.array_nullable now reach the existing Bun/GORM writer options for convert/merge/split output paths. - templ accepts the explicit templ-only output.format: text marker for arbitrary text output; omitted format remains compatible. - Added regression tests for Bun sqltypes preservation and templ text validation/execution, plus documentation/example updates. Changed paths: - pkg/jobs/jobs.go - cmd/relspec/job.go - cmd/relspec/job_test.go - docs/JOB_FILES.md - examples/jobs/relspec.reports.yml Verification: - go test ./pkg/jobs ./cmd/relspec: PASS - go build -o /home/hermes/.tmp/relspec-issue-29 ./cmd/relspec: PASS - go vet ./...: PASS - git diff --check: PASS - go test ./...: BLOCKED by pre-existing environment failure in pkg/writers/dctx TestWriter_WriteSchema: open /tmp/test.dctx: permission denied - make lint: BLOCKED because installed golangci-lint v1 rejects repository golangci-lint v2 configuration Model: gpt-5.6-luna via Hermes openai-codex fallback because Claude spend limit. Blockers: full suite and lint blockers above; no implementation blocker.
sgcommand commented 2026-09-21 04:03:32 +00:00
Member
Copy Link
Copy Source

Resolved by merged PR #30.

  • Branch: issue-29-job-manifest-options
  • Implementation SHA: ade71b75fa5a4b5e37e3e824ae027163348be1d9
  • PR: #30

The merged implementation adds job-manifest Bun/GORM writer options (types, array_nullable) and supports templ text output, with regression tests and documentation updates.

Verification recorded in the PR:

  • go test ./pkg/jobs ./cmd/relspec — pass
  • go build -o /home/hermes/.tmp/relspec-issue-29 ./cmd/relspec — pass
  • go vet ./... — pass
  • git diff --check — pass
  • Full-suite/lint blockers were pre-existing environment/tool-version issues, documented in the PR.
Resolved by merged PR #30. - Branch: `issue-29-job-manifest-options` - Implementation SHA: `ade71b75fa5a4b5e37e3e824ae027163348be1d9` - PR: https://git.warky.dev/wdevs/relspecgo/pulls/30 The merged implementation adds job-manifest Bun/GORM writer options (`types`, `array_nullable`) and supports `templ` text output, with regression tests and documentation updates. Verification recorded in the PR: - `go test ./pkg/jobs ./cmd/relspec` — pass - `go build -o /home/hermes/.tmp/relspec-issue-29 ./cmd/relspec` — pass - `go vet ./...` — pass - `git diff --check` — pass - Full-suite/lint blockers were pre-existing environment/tool-version issues, documented in the PR.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
Compat/Breaking
Breaking change that won't be backward compatible
 Kind/Bug
Something is not working
 Kind/Documentation
Documentation changes
 Kind/Enhancement
Improve existing functionality
 Kind/Feature
New functionality
 Kind/Security
This is security issue
 Kind/Testing
Issue or pull request related to testing
Priority
Critical
The priority is critical
Priority
High
The priority is high
Priority
Low
The priority is low
Priority
Medium
The priority is medium
Reviewed
Confirmed
Issue has been confirmed
Reviewed
Duplicate
This issue or pull request already exists
Reviewed
Invalid
Invalid issue
Reviewed
Won't Fix
This issue won't be fixed
Status
Abandoned
Somebody has started to work on this but abandoned work
Status
Blocked
Something is blocking this issue or pull request
Status
Need More Info
Feedback is required to reproduce issue or to continue work
Compat/Breaking
Breaking change that won't be backward compatible
 Kind/Bug
Something is not working
 Kind/Documentation
Documentation changes
 Kind/Enhancement
Improve existing functionality
 Kind/Feature
New functionality
 Kind/Security
This is security issue
 Kind/Testing
Issue or pull request related to testing
Priority
Critical
1
The priority is critical
Priority
High
2
The priority is high
Priority
Medium
3
The priority is medium
Priority
Low
4
The priority is low
Reviewed
Confirmed
1
Issue has been confirmed
Reviewed
Duplicate
2
This issue or pull request already exists
Reviewed
Invalid
3
Invalid issue
Reviewed
Won't Fix
3
This issue won't be fixed
Status
Blocked
1
Something is blocking this issue or pull request
Status
Need More Info
2
Feedback is required to reproduce issue or to continue work
Status
Abandoned
3
Somebody has started to work on this but abandoned work
No labels
Milestone
No items
No Milestone
Assignees
Clear assignees
hein (Hein) root sam sgcommand (SG Command) support warkanum (Warky)
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: wdevs/relspecgo#29