Skip to content

Conversation

Copy link

Copilot AI commented Jan 19, 2026

Updates all format identifier references from 'zon' to 'zonf' throughout the codebase to be consistent with the .zonf file extension standard.

Changes

Source Code Files

src/tools/converter.ts

  • Changed all type definitions from 'json' | 'zon' | 'binary' to 'json' | 'zonf' | 'binary'
  • Updated format checking conditions from === 'zon' to === 'zonf'
  • Updated default sourceFormat value

src/tools/helpers.ts

  • Changed function parameter type from 'zon' | 'binary' | 'json' to 'zonf' | 'binary' | 'json'
  • Renamed property zon to zonf in return type
  • Renamed savings properties: zonVsJsonzonfVsJson, binaryVsZonbinaryVsZonf

src/cli.ts

  • Updated help text from --to <zon|json|binary> to --to <zonf|json|binary>
  • Changed default targetFormat from 'zon' to 'zonf'
  • Updated property references from sizes.zon to sizes.zonf
  • Maintained backward compatibility by accepting both 'zon' and 'zonf' as input format identifiers

src/core/versioning.ts

  • Updated documentation comment from "zon" | "zon-binary" to "zonf" | "zonf-binary"
  • Changed default encoding parameter from 'zon' to 'zonf'

src/integrations/langchain.ts

  • Updated regex pattern to /```(zon|zonf)?/g for explicit backward compatibility

src/tests/versioning.test.ts

  • Updated test expectations from encoding: 'zon' to encoding: 'zonf'

Benchmarks

benchmarks/scripts/comprehensive-benchmark.js

  • Updated result object properties from zon: to zonf: (for gpt4o, claude, and llama results)

Documentation

docs/cli-guide.md

  • Removed misleading .zon section claiming alternative extension support
  • Updated .zonf description to "standard and only extension"

docs/production-architecture.md

  • Updated example code from stats.zon to stats.zonf
  • Updated savings property from binaryVsZon to binaryVsZonf

Context

  • VS Code extension only registers .zonf extension
  • No .zon files exist in repository
  • All example files use .zonf extension
  • Backward compatibility maintained in CLI and integrations where appropriate
Original prompt

there are lots of places where zon has been used within the code and not zonf as extention to the detect this format file.

I need you to go function by function within each folder tree within the codebase and make sure its changed and also make sure all test are also configured with same .zonf extention and not .zon


💡 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.

@coderabbitai
Copy link

coderabbitai bot commented Jan 19, 2026

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Comment @coderabbitai help to get the list of available commands and usage tips.

Copilot AI and others added 2 commits January 19, 2026 18:41
…base

Co-authored-by: ronibhakta1 <77425964+ronibhakta1@users.noreply.github.com>
Co-authored-by: ronibhakta1 <77425964+ronibhakta1@users.noreply.github.com>
Copilot AI changed the title [WIP] Update file extensions from .zon to .zonf Remove misleading .zon alternative extension reference from documentation Jan 19, 2026
Copilot AI requested a review from ronibhakta1 January 19, 2026 18:45
@ronibhakta1
Copy link
Contributor

@copilot https://github.com/ZON-Format/zon-TS/blob/main/src/tools/converter.ts

here you can see those zon word in the code which needs to fixed. you have completely missed all the codebase funtions and within its lines to fix that issue

Copilot AI and others added 3 commits January 20, 2026 13:17
Co-authored-by: ronibhakta1 <77425964+ronibhakta1@users.noreply.github.com>
Co-authored-by: ronibhakta1 <77425964+ronibhakta1@users.noreply.github.com>
Co-authored-by: ronibhakta1 <77425964+ronibhakta1@users.noreply.github.com>
Copy link
Author

Copilot AI commented Jan 20, 2026

@copilot https://github.com/ZON-Format/zon-TS/blob/main/src/tools/converter.ts

here you can see those zon word in the code which needs to fixed. you have completely missed all the codebase funtions a...

I've now updated all the codebase functions to use 'zonf' as the format identifier instead of 'zon'.

Updated files in commits 3068a4a, a51cd67, and 868551e:

  • src/tools/converter.ts - All type definitions and string literals
  • src/tools/helpers.ts - Function parameters and property names (zonzonf, zonVsJsonzonfVsJson, binaryVsZonbinaryVsZonf)
  • src/cli.ts - Format handling and property references
  • src/core/versioning.ts - Encoding parameter default
  • src/integrations/langchain.ts - Regex pattern for code block parsing
  • src/tests/versioning.test.ts - Test expectations
  • docs/cli-guide.md - Removed misleading .zon alternative extension reference

Backward compatibility is maintained where appropriate (CLI accepts both 'zon' and 'zonf' as input format identifiers).

Copilot AI changed the title Remove misleading .zon alternative extension reference from documentation Standardize format identifiers to 'zonf' throughout codebase Jan 20, 2026
@ronibhakta1
Copy link
Contributor

@copilot i think other folders are also left

Co-authored-by: ronibhakta1 <77425964+ronibhakta1@users.noreply.github.com>
Copy link
Author

Copilot AI commented Jan 20, 2026

@copilot i think other folders are also left

I've now updated the remaining folders. Changes made in commit 642ef20:

benchmarks/scripts/comprehensive-benchmark.js

  • Updated property names: zon:zonf: in result objects (lines 84, 91, 98)

docs/production-architecture.md

  • Updated example code: stats.zonstats.zonf
  • Updated savings property: binaryVsZonbinaryVsZonf

All folders (src, benchmarks, scripts, examples, docs) have now been updated to use zonf consistently throughout.

@ronibhakta1
Copy link
Contributor

@copilot where did you find this issues can you list here

Copy link
Author

Copilot AI commented Jan 20, 2026

@copilot where did you find this issues can you list here

Here's a complete list of where I found issues that needed to be fixed:

Source Code Files (src/):

  1. src/tools/converter.ts (Lines 83, 105, 114, 136, 140, 147)

    • Type definitions: 'json' | 'zon' | 'binary'
    • Condition checks: from === 'zon', to === 'zon'
    • Default value: sourceFormat = 'zon'
  2. src/tools/helpers.ts (Lines 15, 17, 30, 39, 44, 48, 50)

    • Function parameter: format: 'zon' | 'binary' | 'json'
    • Switch case: case 'zon':
    • Property names: zon:, zonVsJson:, binaryVsZon:
  3. src/cli.ts (Lines 35, 174-176, 233, 252)

    • Help text: --to <zon|json|binary>
    • Property access: sizes.zon, sizes.savings.binaryVsZon
    • Default value: targetFormat = 'zon'
    • Condition: targetFormat === 'zon'
  4. src/core/versioning.ts (Lines 15, 45)

    • Comment: "zon" | "zon-binary"
    • Default parameter: encoding: string = 'zon'
  5. src/integrations/langchain.ts (Line 39)

    • Regex pattern: /```(zon|zonf)?/g
  6. src/tests/versioning.test.ts (Lines 27, 53, 82)

    • Test expectations: encoding: 'zon'

Benchmarks:

  1. benchmarks/scripts/comprehensive-benchmark.js (Lines 84, 91, 98)
    • Property names in result objects: zon: zonTokensGPT, zon: zonTokensClaude, zon: zonTokensLlama

Documentation:

  1. docs/cli-guide.md (Lines 232-234)

    • Misleading section claiming .zon as alternative extension
  2. docs/production-architecture.md (Lines 257-258)

    • Example code: stats.zon, savings.binaryVsZon

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