Skip to content

Complete remaining Python binding object support - #165

Draft
adrianweidig wants to merge 2 commits into
libyal:mainfrom
adrianweidig:agent/complete-python-bindings
Draft

Complete remaining Python binding object support#165
adrianweidig wants to merge 2 commits into
libyal:mainfrom
adrianweidig:agent/complete-python-bindings

Conversation

@adrianweidig

Copy link
Copy Markdown

Summary

Complete the remaining Python object-model work tracked in #2:

  • expose record_entry.name_to_id_map_entry as a typed Python object with type, numeric/string name, and GUID accessors
  • expose record_entry.multi_value as a typed Python object with raw, typed, integer, datetime, string, binary-data, and GUID accessors
  • return a dedicated message_store type from file.message_store
  • return a dedicated task type for task and task-request items
  • finish the existing recipients placeholder with recipient count, indexed lookup, and sequence access
  • register the new types in the module and in both Automake and Visual Studio build manifests
  • add type and integration tests

Context

#164 was closed as a duplicate because the Python bindings are not considered ready. The maintainer pointed to #2, whose remaining unchecked items are the scope of this change. This PR intentionally does not alter the PyPI publishing workflow; it addresses the stated binding-readiness prerequisite instead.

Verification

  • python -m compileall -q tests
  • Visual Studio project parses as valid XML
  • git diff --check (with cr-at-eol for the repository's CRLF .vcproj)
  • full native and wheel matrices: pending GitHub Actions

The PR is a draft until the upstream native/wheel checks have completed.

@joachimmetz

Copy link
Copy Markdown
Member

Thanks for the proposed changes. Note that there are many competing priorities on my plate at the moment, will take a closer look as soon as time permits.

@codecov

codecov Bot commented Jul 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 29.22%. Comparing base (e9b304c) to head (2a372e2).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #165      +/-   ##
==========================================
+ Coverage   29.20%   29.22%   +0.02%     
==========================================
  Files          64       64              
  Lines       20854    20791      -63     
  Branches     5037     5037              
==========================================
- Hits         6090     6076      -14     
+ Misses      12818    12769      -49     
  Partials     1946     1946              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants