feat: complete Runic Ward defense layer#2036
Closed
jay9297 wants to merge 45 commits into
Closed
Conversation
- Add scripts/ai-agent.sh with 2-tier fallback (Claude Pro → OpenCode Go) - Add CLAUDE.md and AGENTS.md with project-specific agent instructions - Add .github/workflows: ci.yml, ai-fix.yml, ai-review.yml, auto-merge.yml - Model slugs verified against opencode v1.15.7: review: opencode-go/glm-5.1 fix: opencode/deepseek-v4-flash-free → opencode-go/qwen3.6-plus → opencode-go/deepseek-v4-flash
- .busted: enable coverage = true - .luacov: configure include/exclude paths and report file - .luacheckrc: lua51+luajit std, ignore unused args/loop vars - Dockerfile: install luacheck via luarocks
Remove the build step (pre-built image from GHCR needs no local build) and replace all `tests` service references with `busted-tests` to match the name defined in docker-compose.yml.
docker compose run args replace 'command' but append to 'entrypoint', so --tags and --coverage flags were being treated as executables.
Updated AI fix workflow to extract issue metadata and improve branch handling.
Added concurrency control to cancel in-progress AI reviews and improved diff handling.
Added caching and image management steps to the verify job.
Updated AI fix workflow to improve error handling and timeout settings.
Increased timeout for AI review job from 15 to 30 minutes and updated review prompt structure.
Update commit message format and error handling in AI workflow.
- Make 'rating' optional in evasion suffix patterns (r?a?t?i?n?g?) - Remove dead-code keystone pattern with wrong parameter order - Add bounds check for hollowPalmAddedPhys table (levels 1-40) - Add descriptive line patterns for Hollow Palm Technique node - Add Spectral Ward pattern for 'per X item evasion on equipped body armour' - Update ModCache entries for all affected patterns
* Chore: Bump action version numbers (PathOfBuildingCommunity#1993) * Add support for Legacy Grand Spectrum mods (PathOfBuildingCommunity#1994) Some old mods don't require the socketed in text Co-authored-by: LocalIdentity <localidentity2@gmail.com> * Add support for Ancestral Empowerment, Ancestral Call, Crescendo II and III, Infernal Cry, and "Ancestrally Boosted Attacks" passive nodes (PathOfBuildingCommunity#1839) * add support for Ancestral Call, Crescendo II and III, ancestral boost tree nodes * fix ancestral call inc dmg from tree * big ole refactor so we stop duplicating code one commit so I can revert if it goes boom * added test update CalcSection from "Exerted Warcries" to "Ancestral Boosts" * add data map for gemName given a modSource * boosted attacks configOption * add support for Ancestral Empowerment, including combining Ancestral Empowerment with Fist of War fix Fist of War/ancestrally boosted slams more AOE to inc AOE add test for Combined logic for Ancestral Empowerment and Fist of War * comment spelling * better label * fix avg aoe from fist of war * forgot AOE portion for Crescendo III * add support for Infernal Cry's empowered buff * refactor away from specific baseMod for FistOfWar fix Ancestral Empowerment not getting the double from fist of war III * empowered condition for all Ancestrally Boosted * ancestralBoostEffect * multiplier instead * pr comments also refactored Crescendo III to use the new logic instead of an extra isolated block of logic by introducing a way to conditionally max uptime for Boosted or Empowered attacks * Fix issues Move the Infernal Cry buff to calc offence so it doesn't rely on stale calc output Fix Ancestral Empowerment uptime calculation Move gain table so Infernal Cry still works * Fix spelling --------- Co-authored-by: LocalIdentity <localidentity2@gmail.com> * Add support for 0.5 JSON skill tree (PathOfBuildingCommunity#1984) * Passive tree parser from ggg web data * enhanced escapeGGGString * Fix ascendancy unlock constraint check in node search parameters * Refactor DrawQuadAndRotate to handle different data lengths for vertex calculations * Add node overlay for Blighted Notable Frames in passive tree * Fix typos and improve comments in passive tree script * Fix typos in ascendancy handling and update inner radius calculations in passive tree script * Refactor passive tree node replacement logic and add switchable options for Druid nodes * Add support for alias notable maps and enhance jewel socket handling - Enhanced passivetree_ggg.lua to support aliasNotableMap for jewel sockets. * Rename aliasNotableMap to aliasPassiveSocket for clarity in passive tree data * Enhance jewel socket handling by adding noRadius property and updating related logic in passive tree * Enable Abyssal Lich parser * Fix tests Export ModCache Fix crash with class Ids Fix unarmed data for Ids * Add legacy class ID remap Old builds from 0.1 to 0.3 used the wrong id for the character class and opening them now would convert them to the wrong class. This maps the old builds so they now open correctly * Formatting + typos * AliasPassiveSocket fix Was using the wrong name from a previous commit * Export fixes Uses the tree data folder to look for the ggg assets and json files Fixes the tree file changing each time on export due to pairs usage * Use copy file command to speed up export Now runs in 0.1s instead of 8s --------- Co-authored-by: justjuangui <servicios@juacarvajal.com> Co-authored-by: LocalIdentity <localidentity2@gmail.com> * Remove aliasPassiveSocket temp code * Fix Hollow Palm Technique parsing and add Spectral Ward support - Make 'rating' optional in evasion suffix patterns (r?a?t?i?n?g?) - Remove dead-code keystone pattern with wrong parameter order - Add bounds check for hollowPalmAddedPhys table (levels 1-40) - Add descriptive line patterns for Hollow Palm Technique node - Add Spectral Ward pattern for 'per X item evasion on equipped body armour' - Update ModCache entries for all affected patterns --------- Co-authored-by: Nighty <Nightblade@users.noreply.github.com> Co-authored-by: LocalIdentity <31035929+LocalIdentity@users.noreply.github.com> Co-authored-by: LocalIdentity <localidentity2@gmail.com> Co-authored-by: Peechey <92683202+Peechey@users.noreply.github.com> Co-authored-by: Juangui <80857657+justjuangui@users.noreply.github.com> Co-authored-by: justjuangui <servicios@juacarvajal.com> Co-authored-by: jay9297 <jay9297@users.noreply.github.com>
# Conflicts: # spec/System/TestSkills_spec.lua # src/Data/ModCache.lua # src/Export/Scripts/passivetree_ggg.lua # src/TreeData/0_5/tree.lua
- Regenerate ModCache.lua to match current ModParser (removes stale combined modifier entries from pre-upstream Hollow Palm changes) - Fix spellcheck workflow: remove stale dictionary cache that was causing upstream words to be missed, align with upstream config
- Never open PRs against upstream - Default push target is fork (remote.pushDefault=fork) - Upstream sync via local merge, not PR
Fix typo in specialModList: the GGPK stat ID prefix was 'cover_x%_' but the real stat ID is 'recover_x%_of_maximum_ward_on_persistent_minion_death'. Also add display-text pattern so customMods input can parse this modifier. Add TestWard_spec.lua with 13 tests covering Ward mechanics.
Add Ward pool computation (base/inc/more), WardRegen from base regen rate, WardRechargeDelay, WardBypass, WardRecoverOnBlock, and ward config conditions (LowWard, FullWard) to CalcDefence. Add sidebar display entries for Ward stats in BuildDisplayStats. Add Ward breakdown panel rows in CalcSections. Add WardRechargeDelay and WardRegenRatePercentPerMinute constants to Misc.lua. Add Ward-related config options (conditionLowWard, conditionFullWard, etc).
Add SkillStatMap entries for base_maximum_ward, maximum_ward_+%, ward regeneration, ward cost efficiency, recover_x_ward_on_block, and recover_x_ward_on_charm_use. Add corresponding stat description entries for all ward-related stat IDs.
Strip "Runeforged " prefix from item base names in Item.lua so that Runeforged variants resolve to their underlying base type. Fix ImportTab to handle Runic Ward import correctly.
…ArmourBreak) - Add ward_regeneration_rate_+% and ward_rune_maximum_ward_+%_final stat-ID mappings to SkillStatMap - Route WardRegen into TotalNetRegen and ComprehensiveTotalNetRegen with breakdown display - Apply RuneWardDamageTaken multiplier in ward absorption formula so runic ward drains faster per point absorbed - Add WardRecoverOnBlock to EHP GainWhenHit loop including both restore blocks and accelerated damage forwarding - Route WardAttackHitPercent into ArmourBreakPerHit in CalcOffence - Add test for WardRegen computation
…Regen test - BuildDisplayStats.lua: WardBypass/WardRecoverOnBlock/WardCoverOnMinionDeath sidebar entries already present - CalcSections.lua: Ward breakdown rows (Bypass, Recover on Block/Charm/Minion Death, Rune stats) already present - TestWard_spec.lua: rename test and add nil-safe TotalNetRegen >= WardRegen assertion
…alNetRegen test - ModParser: rename ward_rune_maximum_ward_ to ward_rune_maximum_ward_+%_final and change INC to MORE (_final suffix always denotes a MORE multiplier in GGPK) - CalcDefence: read WardRegen BASE sum so flat per-minute bonuses are no longer silently ignored; convert from per-minute to per-second and include in breakdown - TestWard: replace non-falsifiable TotalNetRegen assertion (== nil or ...) with a degen-triggered test that guarantees TotalNetRegen is computed and correct; add new test verifying Ward MORE stacks multiplicatively with INC
Member
|
I think you didn't mean to open this here, given the AGENTS.md instructions |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
WardCoverOnMinionDeathstat-ID bug (cover_→recover_) in ModParserward_rune_maximum_ward_+%_finalkey truncation and wrong modifier type (INC → MORE)/60unit conversion from per-minute to per-second)TotalNetRegenandComprehensiveTotalNetRegenRuneWardDamageTakenmultiplier in ward absorption formula (ward drains faster per hit)WardRecoverOnBlockwired into EHP GainWhenHit simulation (all 5 touchpoints)WardAttackHitPercentfeedsArmourBreakPerHitin CalcOffenceward_regeneration_rate_+%,ward_rune_maximum_ward_+%_final)Runeforgedprefix from base items on paste/import; normalizeRunic Wardproperty in importTest plan
podman run --rm -e HOME=/tmp -v "$(pwd)":/workdir:ro -w /workdir ghcr.io/pathofbuildingcommunity/pathofbuilding-tests:latest busted --lua=luajit→ 380 successes / 0 failuresRuneforged Xitem into Items tab — base name strips correctly