Skip to content

refactor: rename safePushData to pushDataWithSchemaRepair - #6

Merged
metalwarrior665 merged 1 commit into
masterfrom
claude/rename-safepushdata-85gdm4
Jul 27, 2026
Merged

refactor: rename safePushData to pushDataWithSchemaRepair#6
metalwarrior665 merged 1 commit into
masterfrom
claude/rename-safepushdata-85gdm4

Conversation

@metalwarrior665

@metalwarrior665 metalwarrior665 commented Jul 27, 2026

Copy link
Copy Markdown
Member

Summary

Rename the main export and all related types from safePushData to pushDataWithSchemaRepair for better clarity and consistency across the codebase.

Changes

  • Function rename: safePushData()pushDataWithSchemaRepair()
  • Type renames:
    • SafePushDataResultPushDataWithSchemaRepairResult
    • SafePushDataOptionsPushDataWithSchemaRepairOptions
  • File renames:
    • src/safePushData.tssrc/pushDataWithSchemaRepair.ts
    • test/safePushData.test.tstest/pushDataWithSchemaRepair.test.ts
  • Updated all references: Function calls, imports, log messages, and documentation throughout the codebase
  • Updated exports: Modified index.ts to export the renamed function and types
  • Updated documentation: README and inline comments now reference the new name

Implementation Details

  • All 53 test cases updated to use the new function name; the suite passes unchanged otherwise
  • Console log messages updated to reference pushDataWithSchemaRepair instead of safePushData
  • CI check script updated to reference the new name in comments
  • No functional changes to the implementation logic

Notes

  • DroppedItem, ValidationError, PushFn, and isSchemaValidationError keep their names — they are not specific to the renamed function.
  • Historical CHANGELOG.md entries still say safePushData; they record commits that shipped under the old name and were left untouched.
  • This is a breaking change for consumers of the published package.

https://claude.ai/code/session_012iFDPXYxFwFGUe4PEwSAj7

…thSchemaRepair

Rename the exported function, its option/result types, the source and test
files, and every doc/log/comment mention so the name says what the wrapper
actually does: push, and repair against the dataset schema.

- safePushData -> pushDataWithSchemaRepair
- SafePushDataResult -> PushDataWithSchemaRepairResult
- SafePushDataOptions -> PushDataWithSchemaRepairOptions
- src/safePushData.ts -> src/pushDataWithSchemaRepair.ts
- test/safePushData.test.ts -> test/pushDataWithSchemaRepair.test.ts

Log-line prefixes change with the name, and the tests asserting on them were
updated to match. Historical CHANGELOG entries are left as-is.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012iFDPXYxFwFGUe4PEwSAj7
@metalwarrior665 metalwarrior665 changed the title Rename safePushData to pushDataWithSchemaRepair refactor: rename safePushData to pushDataWithSchemaRepair Jul 27, 2026
@metalwarrior665
metalwarrior665 merged commit 275e904 into master Jul 27, 2026
14 of 15 checks passed
yaroslav-tykhovetskyi-apify added a commit that referenced this pull request Jul 27, 2026
Resolves conflicts from master's safePushData -> pushDataWithSchemaRepair
rename (#6): our vitest-ported test file and CHANGELOG unreleased section
adjusted to match. Also drops gteam-internal from the README's project
structure diagram (its docs already moved to src/gteam-internal/README.md).
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.

3 participants