refactor!: remove not particularly useful features

[muktihari]

After 1 year of using this SDK to create programs. I find that some features are not particularly useful such as (Message) WithMessages andfactory.CreateMesgOnly, we can just directly assign the value. Some of them can be omitted when certain features do not exist to begin with such as internal/kit package. I find some abstractions are a bit confusing such as Reserved and Architecture fields in proto.Message, these fields are just copied from proto.MessageDefinition during decoding and this does not add any value to my program.

These features/values were created at the time when I still didn't know the proper way to implementation of FIT protocol. Now I think these can be simplified.

This PR cleaning up those elements to help us maintain cleaner code in the future.

What's changed with breaking changes:

1. Behavior

decoder's Decode() and DecodeWithContext() are no longer required to be wrapped inside Next() loop, these methods can be invoked multiple times to decode chained FIT file directly. Method Next() is now just a convenience tool to do iteration.

2. Modified

Functions

• proto.CreateMessageDefinition -> proto.NewMessageDefinition
• proto.CreateMessageDefinitionTo -> (*Encoder) newMessageDefinition

Methods

• (FileHeader) MarshalAppend -> (*FileHeader) MarshalAppend
• (MessageDefinition) MarshalAppend -> (*MessageDefinition) MarshalAppend
• (Message) MarshalAppend(n []byte) -> (*Message) MarshalAppend(b []byte, arch byte)

Structs or Fields

• proto: FileHeader.ProtocolVersion type is changed from byte to proto.Version for consistency.
• decoder: AccumulatedValue is now private with the new name value
• decoder: Accumulator's Field: AccumulatedValues is now private with new name values

Errors

• encoder: ErrNotAFitFile -> ErrNotFITFile

3. Removed

Package

• internal/kit

Functions

• factory.CreateMesgOnly

Methods

• (*Factory) CreateMesgOnly
• (proto.FIT) WithMessages
• (proto.Message) WithFields
• (proto.Message) WithDeveloperFields
• (proto.Message) WithFieldValues
• (proto.Message) Clone
• (proto.Field) Clone
• (proto.SubField) Clone
• (proto.DeveloperField) Clone
• (FileHeader) MarshalBinary
• (MessageDefinition) MarshalBinary
• (Message) MarshalBinary

Constants

• proto: MaxBytesPerMessage and MaxBytesPerMessageDefinition are removed

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (c1bf3fa) to head (1961084).

Additional details and impacted files

@@            Coverage Diff            @@
##            master      #418   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           41        41           
  Lines         3695      3612   -83     
=========================================
- Hits          3695      3612   -83     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.