Skip to content

Fix required tag updates for policy tag lists#75

Draft
Tony-MK wants to merge 1 commit intoTags-Violationsfrom
codex/fix-errors-in-policytagtest
Draft

Fix required tag updates for policy tag lists#75
Tony-MK wants to merge 1 commit intoTags-Violationsfrom
codex/fix-errors-in-policytagtest

Conversation

@Tony-MK
Copy link
Owner

@Tony-MK Tony-MK commented Jan 15, 2026

Motivation

  • Fix failures when toggling the policy requiresTag flag that occurred due to attempting to spread or merge undefined tag list entries when building updated policy tag data.

Description

  • Changed getUpdatedTagsData in src/libs/actions/Policy/Tag.ts to iterate over policyData.tags ?? {} and guard against missing tag lists by skipping undefined entries and merging from the actual tagList instead of spreading possibly-undefined accumulator entries.

Testing

  • Attempted to run npm test -- --runTestsByPath tests/actions/PolicyTagTest.ts, but jest was not available in the environment so automated tests could not be executed.

Codex Task

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant