We left a lot of functions in hardwareSerial.cpp in that allow people to use a standard hardware serial. Many of these could probably be removed unless there is another hardware serial BeanUsers could use. Check whether there is, and if not, restructure code to remove never-to-be-used hardware serial virtual functions that eat up code space.

I tried to compile a sketch using ArduinoJson Library and got a compile error. I filed a issue #142 with ArduinoJson, and the response was that there is an issue with Bean String class. See linked issue for complete response.

Thanks,
Brett

Move line 43 closer to 136
Add comments around sleep.
Verify that analog comparator code does what we think it does.

BeanMidi.enable() must be called to add the BLE MIDI service to the bean's advertisements, but neither of the two BeanMidi examples call it:

https://github.com/PunchThrough/bean-arduino-core/blob/master/examples/profiles/MIDI.ino
https://punchthrough.com/bean/docs/guides/features/midi/

The bean's MIDI service needs to be advertised in order for it to appear in the list of available BLE MIDI devices presented to the user by iOS and Android apps.

Suggested fix: Call BeanMidi.enable() in the examples' setup function.

You can quite easily provide support for all platforms this way (Windows, OS X, Linux).

Println() functions send two messages from the Arduino to the CC, one for the message and one for the newline. This should be fixed.

BeanMidi.loadMessage(buf) return value on buffer full is off by 3.

If full, return the current index. The index should only be incremented if the load was successful.

After installing Bean CLI, I'm wondering why I get this error when pressing Upload button in Arduino.

Arduino: 1.6.12 (Mac OS X), TD: 1.36, Board: "LightBlue Bean (2.0.3)"

Sketch uses 5,822 bytes (18%) of program storage space. Maximum is 32,256 bytes.
Global variables use 670 bytes of dynamic memory.
Traceback (most recent call last):
  File "/Applications/Arduino.app/Contents/Java/hardware/tools/bean/post_compile", line 94, in <module>
    copy_hex(parsed_args)
  File "/Applications/Arduino.app/Contents/Java/hardware/tools/bean/post_compile", line 89, in copy_hex
    shutil.copyfile(tmp_compiled_path, perm_compiled_path)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/shutil.py", line 83, in copyfile
    with open(dst, 'wb') as fdst:
IOError: [Errno 13] Permission denied: '/Users/jc/.beansketches/bean/lightblue-bean-test-2-photores.hex'
IOError: [Errno 13] Permission denied: '/Users/jc/.beansketches/bean/lightblue-bean-test-2-photores.hex'

Is there a way to increase permissions of these scripts triggered from inside Arduino app so they can write the files?

Even simple Blink example fails with this error:

In file included from /data2/arduino-1.6.12/hardware/LightBlue-Bean/avr/cores/bean/Arduino.h:193:0,
                 from /tmp/arduino_build_41733/sketch/Blink.ino.cpp:1:
/data2/arduino-1.6.12/hardware/LightBlue-Bean/avr/cores/bean/HardwareSerial.h:30:51: fatal error: applicationMessageHeaders/AppMessages.h: No such file or directory
 #include "applicationMessageHeaders/AppMessages.h"

Arduino-1.6.12
Ubuntu 16.10 (Yakkety Yak)
bean --version: 0.6.0

looks like applicationMessageHeaders is missing in the package.

See Bean.cpp lines:

• 579
• 591
• 603

These TODOs were added in April 2014 and have not been touched since. Is it time to remove them?