Skip to content

Add application step commands and finalize validation - #40

Merged
luis0ares merged 2 commits into
ProgramadoresSemPatria:mainfrom
davidalecrim1:feat/cli-new-commands
May 22, 2026
Merged

luis0ares merged 2 commits into
ProgramadoresSemPatria:mainfrom
davidalecrim1:feat/cli-new-commands

Conversation

@davidalecrim1

@davidalecrim1 davidalecrim1 commented May 19, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Add CLI support for application step management and finalization, enforce backend validation for final outcome combinations so invalid Offer/Accepted states do not reach the database, and add first-class OpenCode skill installation support.

Type of Change

  • Feature
  • Fix
  • Improvement
  • Refactor
  • Documentation

Included Changes

Features

  • Add applika applications steps list/add/edit/delete and applika applications finalize
  • Add JSON output for step listing and finalization, plus applika version
  • Add ~/.agents/skills as a built-in applika skill install target for OpenCode

Improvements

  • Add explicit --application-id and --step-record-id aliases while keeping positional IDs backward compatible
  • Enforce backend finalization rules for Offer plus Accepted and salary_offer
  • Expand CLI and backend test coverage for the new flows and guards

Testing

  • Unit tests
  • Integration tests
  • Manual testing
  • N/A

Notes

  • Backend integration tests could not be run locally in this session because Testcontainers could not reach the Docker daemon.

Checklist

  • Self-review completed
  • CI passing
  • Ready to merge

@italo-gouveia italo-gouveia left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@luis0ares
luis0ares merged commit 6b32793 into ProgramadoresSemPatria:main May 22, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants