Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 8, 2026

  • Fix TAR header formatting errors in package_skill.mjs
    • Change file size field from ' ' (quote space quote) to (space character)
    • Change mtime field from ' ' (quote space quote) to (space character)
    • Verify TAR format compliance with specification
  • Reply to comment with fix details

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Replace incorrect `' '` syntax with proper space character ` ` for
file size and mtime fields. According to TAR specification, these
12-byte octal fields should be terminated with either NUL or space.
The previous code had quote characters inside the template literal
which would have been written literally instead of as terminators.

Co-authored-by: fbeaudoincoveo <23503066+fbeaudoincoveo@users.noreply.github.com>
@fbeaudoincoveo fbeaudoincoveo marked this pull request as ready for review January 8, 2026 15:32
@fbeaudoincoveo fbeaudoincoveo requested a review from a team as a code owner January 8, 2026 15:32
@fbeaudoincoveo fbeaudoincoveo merged commit 0341fa1 into KIT-5367 Jan 8, 2026
3 checks passed
@fbeaudoincoveo fbeaudoincoveo deleted the copilot/sub-pr-6921 branch January 8, 2026 15:32
Copilot AI requested a review from fbeaudoincoveo January 8, 2026 15:32
Copilot stopped work on behalf of fbeaudoincoveo due to an error January 8, 2026 15:32
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