aethyr/Aethyr
aethyr/Aethyr
2
0
Fork 0
Code Issues 281 Pull requests Projects Releases Packages Wiki Activity Actions

feat(combat): implement 12 one-handed weapon attack actions #332

Merged
freemo merged 4 commits from feature/m4-onehanded-attacks into master 2026-04-01 20:37:36 +00:00
Conversation 0 Commits 4 Files changed 16 +401 -27
freemo commented 2026-03-23 05:49:46 +00:00
Owner
Copy link

Summary

  • Implements 12 one-handed weapon attacks: Quick Slash R, Quick Slash L, Quick Thrust, Slash Right, Slash Left, Overhead Strike, Rising Cut, Cross Slash, Lunging Thrust, Whirlwind, Feint, and Backstab
  • Creates CombatAction class with all 19 properties as specified
  • Creates CombatActionCatalog registry for organizing actions
  • Includes unit and integration tests

Closes #274

## Summary - Implements 12 one-handed weapon attacks: Quick Slash R, Quick Slash L, Quick Thrust, Slash Right, Slash Left, Overhead Strike, Rising Cut, Cross Slash, Lunging Thrust, Whirlwind, Feint, and Backstab - Creates CombatAction class with all 19 properties as specified - Creates CombatActionCatalog registry for organizing actions - Includes unit and integration tests Closes #274
freemo added this to the v1.3.0 milestone 2026-03-23 05:49:46 +00:00
freemo force-pushed feature/m4-onehanded-attacks from 7ab31279ee
Some checks failed
CI / type_check (pull_request) Failing after 8m9s
Details
CI / lint (pull_request) Failing after 15m51s
Details
CI / integration_tests (pull_request) Failing after 15m7s
Details
CI / unit_tests (pull_request) Failing after 15m7s
Details
CI / build (pull_request) Has been skipped
Details
to 06dbbd8024
Some checks failed
CI / integration_tests (pull_request) Failing after 4m4s
Details
CI / lint (pull_request) Successful in 5m48s
Details
CI / type_check (pull_request) Successful in 9m53s
Details
CI / unit_tests (pull_request) Failing after 20m45s
Details
CI / build (pull_request) Has been skipped
Details
2026-03-31 19:45:00 +00:00 Compare
freemo force-pushed feature/m4-onehanded-attacks from 06dbbd8024
Some checks failed
CI / integration_tests (pull_request) Failing after 4m4s
Details
CI / lint (pull_request) Successful in 5m48s
Details
CI / type_check (pull_request) Successful in 9m53s
Details
CI / unit_tests (pull_request) Failing after 20m45s
Details
CI / build (pull_request) Has been skipped
Details
to 27a7309fc4
Some checks failed
CI / type_check (pull_request) Successful in 42s
Details
CI / lint (pull_request) Successful in 1m16s
Details
CI / integration_tests (pull_request) Failing after 4m25s
Details
CI / unit_tests (pull_request) Failing after 22m1s
Details
CI / build (pull_request) Has been skipped
Details
2026-03-31 21:13:41 +00:00 Compare
freemo force-pushed feature/m4-onehanded-attacks from 27a7309fc4
Some checks failed
CI / type_check (pull_request) Successful in 42s
Details
CI / lint (pull_request) Successful in 1m16s
Details
CI / integration_tests (pull_request) Failing after 4m25s
Details
CI / unit_tests (pull_request) Failing after 22m1s
Details
CI / build (pull_request) Has been skipped
Details
to 6244798688
Some checks failed
CI / type_check (pull_request) Successful in 38s
Details
CI / lint (pull_request) Successful in 1m14s
Details
CI / integration_tests (pull_request) Failing after 1m57s
Details
CI / unit_tests (pull_request) Failing after 21m32s
Details
CI / build (pull_request) Has been skipped
Details
2026-04-01 00:37:01 +00:00 Compare
freemo force-pushed feature/m4-onehanded-attacks from 6244798688
Some checks failed
CI / type_check (pull_request) Successful in 38s
Details
CI / lint (pull_request) Successful in 1m14s
Details
CI / integration_tests (pull_request) Failing after 1m57s
Details
CI / unit_tests (pull_request) Failing after 21m32s
Details
CI / build (pull_request) Has been skipped
Details
to a417563e66
Some checks failed
CI / type_check (pull_request) Successful in 45s
Details
CI / lint (pull_request) Successful in 1m19s
Details
CI / integration_tests (pull_request) Failing after 1m29s
Details
CI / unit_tests (pull_request) Failing after 22m0s
Details
CI / build (pull_request) Has been skipped
Details
2026-04-01 01:18:06 +00:00 Compare
fix: restore config.yaml and preserve Combat namespace in test stubs
All checks were successful
CI / type_check (pull_request) Successful in 42s
Details
CI / lint (pull_request) Successful in 1m15s
Details
CI / integration_tests (pull_request) Successful in 4m46s
Details
CI / unit_tests (pull_request) Successful in 21m49s
Details
CI / build (pull_request) Successful in 43s
Details
0492fd50d9 
- Revert config.yaml address from '7' to 127.0.0.1 (fixes server startup)
- Revert config.yaml log_level and start_room to master values
- Fix Combat namespace stubbing in test step definitions to preserve
  existing module contents (CombatAction, ShieldResolver, etc.)
  instead of replacing Aethyr::Core::Combat with an empty stub
- Affected files: slash_steps, kick_command_steps, punch_command_steps,
  simple_block_steps, simple_dodge_steps
freemo merged commit 7f6dceedd4 into master 2026-04-01 20:37:36 +00:00
freemo deleted branch feature/m4-onehanded-attacks 2026-04-01 20:37:36 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
Blocked
Cannot proceed, depends on unresolved issue

  
Duplicate
Issue is a duplicate of an existing one

  
MoSCoW/Could Have
Desirable but not necessary, included if time permits

  
MoSCoW/Must Have
Essential, project cannot ship without it

  
MoSCoW/Should Have
Important but not strictly essential

  
Points/1
Estimated effort: 1 story points

  
Points/13
Estimated effort: 13 story points

  
Points/2
Estimated effort: 2 story points

  
Points/21
Estimated effort: 21 story points

  
Points/3
Estimated effort: 3 story points

  
Points/5
Estimated effort: 5 story points

  
Points/8
Estimated effort: 8 story points

  
Priority/Backlog
Not yet prioritized

  
Priority/Critical
Must be addressed immediately

  
Priority/High
Important, should be completed soon

  
Priority/Low
Nice to have, not time-sensitive

  
Priority/Medium
Normal priority, valid and should be addressed

  
State/Completed
Issue resolved and all PRs merged

  
State/In progress
Someone is actively working on this issue

  
State/In review
Implementation complete, PR submitted for review

  
State/Paused
Work temporarily suspended, typically blocked

  
State/Unverified
Issue created but not yet reviewed by a maintainer

  
State/Verified
Maintainer confirmed valid and actionable

  
State/Wont Do
Deliberate decision not to address this issue

  
Type/Bug
A defect in existing functionality

  
Type/Epic
Large body of work broken into multiple issues

  
Type/Feature
A new capability or user story

  
Type/Legendary
Major strategic pillar composed of multiple Epics

  
Type/Task
Technical or administrative work (not bug or feature)

  
Type/Testing
Test-only work item (TDD, test infrastructure, coverage)

No labels
Blocked
Duplicate
MoSCoW/Could Have
MoSCoW/Must Have
MoSCoW/Should Have
Points/1
Points/13
Points/2
Points/21
Points/3
Points/5
Points/8
Priority/Backlog
Priority/Critical
Priority/High
Priority/Low
Priority/Medium
State/Completed
State/In progress
State/In review
State/Paused
State/Unverified
State/Verified
State/Wont Do
Type/Bug
Type/Epic
Type/Feature
Type/Legendary
Type/Task
Type/Testing
Milestone
Clear milestone
No items
No milestone
v1.3.0
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
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
aethyr/Aethyr!332
No description provided.