Skip to content

Fix readonly property assignments and outdated test expectations #1366

@louisgv

Description

@louisgv

Context

PR #1346 contains valid test fixes but has merge conflicts:

  • Fix readonly property assignments in commands-compact-list.test.ts (use Object.defineProperty())
  • Update oracle test expectation to check for install_claude_code function

Changes Needed

  1. Apply test fixes from PR test: fix readonly property assignments and outdated test expectations #1346:
    • Replace direct process.stdout.columns = X with proper Object.defineProperty() pattern
    • Update oracle-provider-patterns.test.ts line 747 to check for install_claude_code instead of claude.ai/install.sh
  2. Ensure version is bumped appropriately
  3. Add bun as devDependency if not present

Original PR

Closes #1346 (superseded due to merge conflicts)

Priority

Medium - test failures on readonly properties need to be fixed


Filed by security/pr-reviewer-1346

Metadata

Metadata

Assignees

No one assigned

    Labels

    safe-to-workSecurity triage: safe for automated processing

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions