Fix js-typecheck regression in parseAwContext object field access

[Copilot]

Bug Fix

The js-typecheck job failed in push_to_pull_request_branch.cjs because item_type/item_number were read directly from a value narrowed only to object. TypeScript correctly rejected those property reads.

• What was the bug?

  • validateAndNormalizeParsedContext accessed parsedObj.item_type and parsedObj.item_number after an object guard, which is not sufficient for typed property access under @ts-check.
  • This caused TS2339 errors in CI (Property ... does not exist on type 'object').

• How did you fix it?

  • Replaced direct property reads with Reflect.get to read keys from an unknown object safely while preserving runtime behavior.
  • Kept existing normalization semantics:
    • item_type must be a string, otherwise ""
    • item_number is parsed via existing parsePositiveInteger path

const itemTypeValue = Reflect.get(parsed, "item_type");
const itemNumberValue = Reflect.get(parsed, "item_number");
const itemType = typeof itemTypeValue === "string" ? itemTypeValue : "";
const itemNumber = parsePositiveInteger(itemNumberValue);

• Testing

  • actions/setup/js typecheck path is now clean for this file (the failing TS2339 errors are removed).

[pelikhan]

@copilot keep it simple, do not use reflect

[Copilot]

Updated in a2ca378. I removed Reflect.get and switched to a simple typed object cast with direct field access in parseAwContext.

[Copilot]

Pull request overview

Fixes a js-typecheck regression in parseAwContext by avoiding direct property access on values narrowed only to object under @ts-check.

Changes:

• Replace direct reads of item_type / item_number from a loosely-narrowed object with Reflect.get.
• Preserve existing normalization behavior for item_type (string or "") and item_number (via parsePositiveInteger).

Show a summary per file

File	Description
actions/setup/js/push_to_pull_request_branch.cjs	Updates parseAwContext normalization to avoid TS2339 property-access errors under @ts-check.

Copilot's findings

Tip

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

• Files reviewed: 1/1 changed files
• Comments generated: 1

[pelikhan]

@copilot review all comments

[pelikhan]

@copilot lint js https://github.com/github/gh-aw/actions/runs/25860580499/job/75989321699#step:6:1

[Copilot]

@copilot review all comments

Reviewed all comments and updated the implementation in 90b3903 to use Record<string, unknown> with bracket access for item_type/item_number (no Reflect.get).