Add Excel and Word Objects support #6
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
WIP add ThisWorkbook and Sheets support Add Objects Fix string replacement syntax FIx metadata Minor improvement to logging and error handling re: Fix metadata Use trimmed version for Version and Attribute Early exit metadata parsing Add log for count of files re: Fix metadata Add support for Word Document Objects Fix syntax Split code by using functions re: Fix Syntax Remove unnecessary Export-ModuleMember statement for dot-sourced functions
There was a problem hiding this comment.
Pull Request Overview
Adds support for importing Excel and Word VBA objects by introducing test stubs, utility import functions, and integrating object import into the build script.
- Added VBA class stubs for
ThisWorkbook,Sheet1, andThisDocumentin test fixtures. - Created
Object-Import.ps1with parsing, component lookup, and application-specific import logic. - Updated
Build-VBA.ps1to dot-sourceObject-Import.ps1, set up object folders, and invoke the new import workflow.
Reviewed Changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| tests/WordDocument.docm/.../ThisDocument.doc.cls | Added Word object stub for ThisDocument |
| tests/ExcelWorkbook.xlsm/.../ThisWorkbook.wbk.cls | Added Excel workbook object stub |
| tests/ExcelWorkbook.xlsm/.../Sheet1.sheet.cls | Added Excel worksheet object stub |
| tests/ExcelAddin.xlam/.../ThisWorkbook.cls | Added Excel add-in workbook stub |
| tests/ExcelAddin.xlam/.../Sheet1.cls | Added Excel add-in worksheet stub |
| scripts/utils/Object-Import.ps1 | New utility functions for parsing and importing VBA components |
| scripts/Build-VBA.ps1 | Integrated object import, folder setup, and updated dot-sourcing |
Comments suppressed due to low confidence (3)
scripts/Build-VBA.ps1:7
- Typo in 'utiliies'; should be 'utilities'.
# Load utiliies
scripts/utils/Object-Import.ps1:6
- [nitpick] Add comment-based help (e.g., <# .SYNOPSIS, .PARAMETER >) for this function to improve maintainability and clarity.
function Parse-Lines {
tests/ExcelAddin.xlam/Microsoft Excel Objects/ThisWorkbook.cls:1
- [nitpick] File extension '.cls' for ThisWorkbook differs from the '.wbk.cls' pattern used in workbook tests; consider renaming to 'ThisWorkbook.wbk.cls' for consistency.
VERSION 1.0 CLASS
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added support for importing application-specific VBA objects:
ThisWorkbook(Workbook):ThisWorkbook.clsor*.wbk.clsSheet(Worksheet):Sheet\d+.cls(where \d+ matches with any number of digits) or*.sheet.clsThisDocument(Document):ThisDocument.clsor*.doc.clsNote that those objects need to be located in subdirectories
/Microsoft Excel Objectsand/Microsoft Word Objectsrespectively to be imported. This restriction will be removed when we introduce a manifest file.New utility functions in
Object-Import.ps1:Parse-Linesfunction to strip metadata headers from VBA files before importing.Import-CodeToComponent,Find-VbaComponent, and application-specific functions (Import-ExcelObjects,Import-WordObjects) to handle importing.Import-ObjectCodefunction as a main entry point for importing objects based on the Office application type.