Skip to content

Add CI check for the Windows path length limit - #14

Merged
inrayTW1 merged 3 commits into
OPC-Router:mainfrom
JSGInray:feature/check-path-length
Oct 5, 2026
Merged

inrayTW1 merged 3 commits into
OPC-Router:mainfrom
JSGInray:feature/check-path-length

Conversation

@JSGInray

Copy link
Copy Markdown
Contributor

Copilot

OPC Router installs the Solution Gallery below C:\ProgramData\inray\OPC Router 5\config\solution-gallery\<source id>\, so up to 83 characters precede every path of this repository (the source id is 24 characters in current releases and 8 characters after tp#354132). Windows limits a full path to 259 characters, so a path in this repository must stay at or below 176 characters, otherwise the setup cannot write the file.

Change

  • .github/scripts/check-path-length.sh fails with an annotated error for every file path longer than 170 characters (counted from the repository root, .git, .github and img excluded, as in the release zip). The limit can be overridden with MAX_SOLUTION_PATH_LENGTH.
  • New workflow Check path length runs it on pull requests, pushes to main and on demand.
  • Create release runs the same check before building solutions.zip, so an over-long path cannot be released.
  • CONTRIBUTING.md documents the limit.

Current state

The longest path today is 165 characters (OPC UA high throughput streaming to InfluxDB/config/versioned/ConnectionGroups/Connections/Multi Datachange Trigger to InfluxDB/${mdc_to_influx_connection_name}.yaml), so the check passes. Verified locally: passes at the default limit, fails on that file with a limit of 160.

🤖 Generated with Claude Code

The product installs the gallery below
C:\ProgramData\inray\OPC Router 5\config\solution-gallery\<source id>\,
so up to 83 characters precede every path of this repository. A path
over 176 characters cannot be written on Windows and breaks the setup.

The check fails for paths over 170 characters; it runs on pull requests,
on pushes to main and before the release zip is created.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Comment thread CONTRIBUTING.md Outdated
Comment thread .github/scripts/check-path-length.sh Outdated
Comment thread .github/scripts/check-path-length.sh Outdated
JSGInray and others added 2 commits September 23, 2026 16:47
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@JSGInray
JSGInray requested a review from inrayTW1 September 25, 2026 08:10
@inrayTW1
inrayTW1 merged commit d245845 into OPC-Router:main Oct 5, 2026
1 check passed
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