Skip to content

Conversation

@arnoli777
Copy link
Collaborator

Refactors the codesage snapshot create command to align with new user requirements:

  • For Python and Go, the command now produces semantic snapshots in yaml or json format, consistent with the output of the standalone scripts in semantic-snapshot/.
  • The original versioned JSON snapshot functionality is preserved for other languages (e.g., Java, Shell) to avoid regressions.
  • The GoSemanticSnapshotBuilder is optimized to compile its embedded Go AST parser only once per run, fixing a major performance issue.
  • The PythonSemanticSnapshotBuilder is updated to match the detailed output of the reference script.
  • A bug in the snapshot cleanup logic that could cause unintended data loss has been fixed by reverting to the original, safer implementation.
  • Adds quickstart.md to document the updated command usage.
  • The test suite is updated to verify all new and restored functionality, including deep comparisons against the original scripts' output.

Refactors the `codesage snapshot create` command to align with new user requirements:
- For Python and Go, the command now produces semantic snapshots in `yaml` or `json` format, consistent with the output of the standalone scripts in `semantic-snapshot/`.
- The original versioned JSON snapshot functionality is preserved for other languages (e.g., Java, Shell) to avoid regressions.
- The `GoSemanticSnapshotBuilder` is optimized to compile its embedded Go AST parser only once per run, fixing a major performance issue.
- The `PythonSemanticSnapshotBuilder` is updated to match the detailed output of the reference script.
- A bug in the `snapshot cleanup` logic that could cause unintended data loss has been fixed by reverting to the original, safer implementation.
- Adds `quickstart.md` to document the updated command usage.
- The test suite is updated to verify all new and restored functionality, including deep comparisons against the original scripts' output.
@arnoli777 arnoli777 merged commit e7a324e into master Dec 1, 2025
1 check failed
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