ProtoDef-io / ProtoDef-io/node-protodef

Add validate input option

Open
#128 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
JavaScript
Stars
35
Forks
25
Avg merge
4d 3m
Merged PRs (30d)
2

Description

Can be implemented by having a validate function for each datatype

useful to debug calls to write

should be optional to avoid perf issues

examples:

  • i8 : check whether input is between -128 and 127
  • container : check all fields are present and no additional, check all subtypes

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by inspecting the datatype implementations and the write path to find where optional validation can be added. Define how primitive ranges and container fields/subtypes are checked, then verify that validation is disabled by default and that invalid inputs are reported without affecting normal writes.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
api
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.