Make timeout-minutes a templatable integer in schema + custom job compilation

[Copilot]

timeout-minutes needed to validate as a templatable_integer and behave consistently when provided as either a literal integer or a GitHub Actions expression. This change aligns schema validation and Go-side job rendering with that contract.

• Schema updates

  • Switched workflow-level and job-level timeout-minutes definitions to the shared templatable-integer shape in main_workflow_schema.json.
  • Ensures ${{ ... }} timeout expressions validate in the same path as integer literals.

• Go compiler behavior

  • Extended custom job parsing to preserve expression-form timeouts instead of only numeric values.
  • Job YAML generation now emits expression-form timeout-minutes when present.

• Guardrails

  • Rejects non-expression string values for timeout-minutes in custom jobs.
  • Rejects ambiguous job configs that set both integer and expression timeout forms.

• Targeted coverage additions

  • Added parser schema test for templated workflow/job timeout acceptance.
  • Added workflow tests for expression rendering, invalid string rejection, and conflict validation.

jobs:
  build:
    runs-on: ubuntu-latest
    timeout-minutes: ${{ inputs.job_timeout }}
    steps:
      - run: echo "hello"

[Copilot]

Pull request overview

This PR aligns timeout-minutes handling across the JSON schema and the Go workflow compiler so that it accepts either a literal integer or a GitHub Actions expression (${{ ... }}), and renders expression-form timeouts correctly for custom jobs.

Changes:

• Update workflow-level and job-level schema for timeout-minutes to use a shared “templatable integer” shape.
• Extend custom job compilation to preserve expression-form timeout-minutes and emit it during job YAML rendering.
• Add targeted tests covering schema acceptance, rendering of expression timeouts, and rejection of invalid string values / conflicting configs.

Show a summary per file

File	Description
pkg/workflow/jobs.go	Add TimeoutMinutesExpression and render expression-form timeout-minutes when present.
pkg/workflow/jobs_validation.go	Extend validation to forbid timeout-minutes (int or expression) on reusable-workflow caller jobs and reject int+expression conflicts.
pkg/workflow/jobs_test.go	Add unit tests for reusable-workflow restriction with expression timeouts and conflict validation.
pkg/workflow/compiler_jobs.go	Parse custom job timeout-minutes strings as expressions (reject non-expression strings).
pkg/workflow/compiler_jobs_test.go	Add tests for expression parsing/rendering and invalid string rejection.
pkg/parser/schemas/main_workflow_schema.json	Switch workflow/job timeout-minutes to reference templatable_integer.
pkg/parser/schema_test.go	Add schema test ensuring templated workflow/job timeout-minutes values validate.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

• Files reviewed: 7/7 changed files
• Comments generated: 3