Skip to content

[Deprecation] PAL Script Evaluation APIs #343

@silug

Description

@silug

Summary

Two PAL class methods are marked deprecated with replacement APIs and stale “remove in 1.0” messaging.

Evidence

  • lib/puppet/pal/pal_impl.rb:113 marks evaluate_script_string deprecated, replace with with_script_compiler + evaluate_string.
  • lib/puppet/pal/pal_impl.rb:126 marks evaluate_script_manifest deprecated, replace with with_script_compiler + evaluate_file.

Proposed OpenVox 9 Change

  • Remove deprecated class methods:
    • evaluate_script_string
    • evaluate_script_manifest
  • Retain and document modern compiler-block APIs.

Compatibility / Risk

  • Medium risk for API consumers using old PAL entry points.
  • Migration path is clear and already documented in code comments.

Implementation Notes

  • Search internal and test call sites and migrate before deletion.
  • Update API docs and changelog with direct before/after examples.

Acceptance Criteria

  • Deprecated PAL methods are removed.
  • No internal call sites remain.
  • Existing PAL behavior is preserved via supported methods.

Suggested Tests

  • PAL integration tests via with_script_compiler.
  • API-level tests ensuring supported evaluation methods still work for string and file input.

Metadata

Metadata

Assignees

No one assigned

    Labels

    deprecationThis issue pertains to an item/topic which has been deprecated.

    Type

    No type

    Projects

    Status

    Deprecated

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions