-
Notifications
You must be signed in to change notification settings - Fork 3
Standardize format identifiers to 'zonf' throughout codebase #2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the Comment |
…base Co-authored-by: ronibhakta1 <77425964+ronibhakta1@users.noreply.github.com>
Co-authored-by: ronibhakta1 <77425964+ronibhakta1@users.noreply.github.com>
.zon alternative extension reference from documentation
|
@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 |
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>
I've now updated all the codebase functions to use Updated files in commits 3068a4a, a51cd67, and 868551e:
Backward compatibility is maintained where appropriate (CLI accepts both 'zon' and 'zonf' as input format identifiers). |
.zon alternative extension reference from documentation|
@copilot i think other folders are also left |
Co-authored-by: ronibhakta1 <77425964+ronibhakta1@users.noreply.github.com>
I've now updated the remaining folders. Changes made in commit 642ef20: benchmarks/scripts/comprehensive-benchmark.js
docs/production-architecture.md
All folders (src, benchmarks, scripts, examples, docs) have now been updated to use |
|
@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/):
Benchmarks:
Documentation:
|
Updates all format identifier references from
'zon'to'zonf'throughout the codebase to be consistent with the.zonffile extension standard.Changes
Source Code Files
src/tools/converter.ts
'json' | 'zon' | 'binary'to'json' | 'zonf' | 'binary'=== 'zon'to=== 'zonf'src/tools/helpers.ts
'zon' | 'binary' | 'json'to'zonf' | 'binary' | 'json'zontozonfin return typezonVsJson→zonfVsJson,binaryVsZon→binaryVsZonfsrc/cli.ts
--to <zon|json|binary>to--to <zonf|json|binary>'zon'to'zonf'sizes.zontosizes.zonfsrc/core/versioning.ts
"zon" | "zon-binary"to"zonf" | "zonf-binary"'zon'to'zonf'src/integrations/langchain.ts
/```(zon|zonf)?/gfor explicit backward compatibilitysrc/tests/versioning.test.ts
encoding: 'zon'toencoding: 'zonf'Benchmarks
benchmarks/scripts/comprehensive-benchmark.js
zon:tozonf:(for gpt4o, claude, and llama results)Documentation
docs/cli-guide.md
.zonsection claiming alternative extension support.zonfdescription to "standard and only extension"docs/production-architecture.md
stats.zontostats.zonfbinaryVsZontobinaryVsZonfContext
.zonfextension.zonfiles exist in repository.zonfextensionOriginal prompt
💡 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.