Skip to content

loaderDWG: синхронизировать runtime LibreDWG для проверки handle wrap - #1222

Open
konard wants to merge 2 commits into
veb86:masterfrom
konard:issue-1221-7d768d88a4ce
Open

konard wants to merge 2 commits into
veb86:masterfrom
konard:issue-1221-7d768d88a4ce

Conversation

@konard

@konard konard commented May 18, 2026

Copy link
Copy Markdown

Summary

Fixes #1221.

  • Added a loaderDWG. Поиск причины ошибки #1221 section to cad_source/zengine/fileformats/DWG_HANDLE_LOAD_ANALYSIS.md with the new evidence from zcad.log and side files.
  • Identified the strongest current hypothesis: ZCAD runtime used an older x86_64-win64 libredwg-0.dll while fpdwginspect used a newer DLL that reads the same DWG correctly.
  • Synced environment/runtimefiles/x86_64-win64/common/bin/libredwg-0.dll to the fpdwginspect LibreDWG binary.

Reproduction / Verification Plan

  1. Build or refresh the ZCAD runtime so environment/runtimefiles/x86_64-win64/common/bin/libredwg-0.dll is copied into cad/bin.
  2. Load the problematic DWG with ZCAD_DWG_DIAG=trace and ZCAD_DWG_TARGET_HANDLES=A325E.
  3. Confirm the raw trace no longer wraps FFFF -> 0 -> 1, and that .dwg.handles.csv contains handles above FFFF.
  4. Confirm duplicate-handle warnings for the post-FFFF sequence disappear.

Local Checks

  • git diff --check
  • make checkvars completed, but this container prints missing tag/Lazarus config noise.
  • Pascal tests were not run locally because this workspace does not provide fpc or lazbuild.

konard added 2 commits May 18, 2026 12:27
Adding .gitkeep for PR creation (default mode).
This file will be removed when the task is complete.

Issue: veb86#1221
@konard konard changed the title [WIP] loaderDWG. Поиск причины ошибки loaderDWG: синхронизировать runtime LibreDWG для проверки handle wrap May 18, 2026
@konard
konard marked this pull request as ready for review May 18, 2026 12:37
@konard

konard commented May 18, 2026

Copy link
Copy Markdown
Author

Working session summary

Implemented and pushed the update to PR 1222: #1222

Changed:

Verification:

  • git diff --check passed.
  • make checkvars completed, with existing container noise about missing tag/Lazarus config.
  • Pascal tests were not run because this workspace has no fpc or lazbuild.
  • No GitHub Actions runs/checks are currently reported for this branch.

This summary was automatically extracted from the AI working session output.

@konard

konard commented May 18, 2026

Copy link
Copy Markdown
Author

🤖 Solution Draft Log

This log file contains the complete execution trace of the AI solution draft process.

💰 Cost estimation:

  • Model: GPT-5.5
  • Provider: OpenAI
  • Public pricing estimate: $8.200771

📊 Context and tokens usage:

  • 174.5K / 1.1M (17%) input tokens, 24.5K / 128K (19%) output tokens

Total: (174.5K + 5.4M cached) input tokens, 24.5K output tokens, $8.200771 cost

🤖 Models used:

  • Tool: OpenAI Codex
  • Requested: gpt-5.5
  • Model: GPT-5.5 (gpt-5.5)

📎 Log file uploaded as Gist (22596KB)


Now working session is ended, feel free to review and add any feedback on the solution draft.

@konard

konard commented May 18, 2026

Copy link
Copy Markdown
Author

✅ Ready to merge

This pull request is now ready to be merged:

  • CI workflows exist but were not triggered for this commit
  • No merge conflicts
  • No pending changes

Monitored by hive-mind with --auto-restart-until-mergeable flag

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.

loaderDWG. Поиск причины ошибки

1 participant